Skip to content

Facets across the rest of the resources#441

Draft
thokra-nav wants to merge 7 commits into
mainfrom
facets
Draft

Facets across the rest of the resources#441
thokra-nav wants to merge 7 commits into
mainfrom
facets

Conversation

@thokra-nav
Copy link
Copy Markdown
Contributor

No description provided.

This comment was marked as outdated.

This comment was marked as outdated.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 60 out of 73 changed files in this pull request and generated 4 comments.

Files not reviewed (13)
  • internal/graph/gengql/applications.generated.go: Language not supported
  • internal/graph/gengql/bigquery.generated.go: Language not supported
  • internal/graph/gengql/bucket.generated.go: Language not supported
  • internal/graph/gengql/complexity.go: Language not supported
  • internal/graph/gengql/config.generated.go: Language not supported
  • internal/graph/gengql/facets.generated.go: Language not supported
  • internal/graph/gengql/jobs.generated.go: Language not supported
  • internal/graph/gengql/kafka.generated.go: Language not supported
  • internal/graph/gengql/opensearch.generated.go: Language not supported
  • internal/graph/gengql/postgres.generated.go: Language not supported
  • internal/graph/gengql/secret.generated.go: Language not supported
  • internal/graph/gengql/teams.generated.go: Language not supported
  • internal/graph/gengql/valkey.generated.go: Language not supported

Comment thread internal/workload/secret/sortfilter.go
Comment thread internal/graph/pagination/models.go Outdated
Comment thread internal/graph/pagination/models.go
Comment thread internal/persistence/postgres/facets.go
Replace EnvironmentFacetItem and similar per-resource environment facet
types
with a shared StringFacetItem type across all facets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants