Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(glean-backend): index all TechDocs-enabled Entities #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

beejiujitsu
Copy link

@beejiujitsu beejiujitsu commented Dec 17, 2024

Jira Ticket

N/A (I don't have access to your internal Notion/Jira)

Why

This is a large PR with a huge refactor / new features I spent months to make production ready.

What is changing

  • feat: batch index all techdocs entities to Glean
  • feat: regularly schedule indexing via config
  • feat: configurable datasource
  • fix getTechDocsMetadataUrl, getTechDocsStaticUrl 404 Not Found (wrong url!)
  • feat: implement getTechDocsEntities using native backstage CatalogApi
  • chore: migrate to new backend system
  • chore: upgrade dependencies
  • refactor: Service -> Client
  • feat: GleanDocument, use entityUri for ID to prevent name collisions across different kinds
  • style: Wealthsimple -> Company
  • fix getTechDocsMetadataUrl, getTechDocsStaticUrl Unauthorized (use authentication)
  • chore: lint

Checklist

How does this PR handle security?

Uses the native backstage AuthService for authenticating against frontend and backend services, where available.

config.d.ts properly marks entries that are secret as such, hiding them from the UI with plugins such as devtools.

Screenshot

Screenshot 2024-12-17 at 9 37 49 AM

Next steps

@beejiujitsu beejiujitsu marked this pull request as ready for review December 17, 2024 14:41
@beejiujitsu beejiujitsu requested a review from a team as a code owner December 17, 2024 14:41
@beejiujitsu beejiujitsu force-pushed the working-glean-backend-indexer branch from 4a96f45 to 56affdc Compare December 17, 2024 15:37
@beejiujitsu beejiujitsu changed the title feat(glean-backend): working indexer feat(glean-backend): TechDocs indexer Dec 17, 2024
@beejiujitsu beejiujitsu changed the title feat(glean-backend): TechDocs indexer feat(glean-backend): index all TechDocs-enabled Entities Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant