Skip to content

feat(aiven-credentials): add activity log GraphQL schema types#363

Merged
jhrv merged 3 commits intomainfrom
fix/aiven-credentials-euthanaisa-label
Mar 17, 2026
Merged

feat(aiven-credentials): add activity log GraphQL schema types#363
jhrv merged 3 commits intomainfrom
fix/aiven-credentials-euthanaisa-label

Conversation

@jhrv
Copy link
Contributor

@jhrv jhrv commented Mar 17, 2026

Summary

  • Adds missing GraphQL schema extensions so activity log entries for Aiven credential creation are exposed via the API
  • Without this, entries were written to the database but would panic if queried via GraphQL (no type mapping in the generated code)
  • Adds AIVEN_CREDENTIALS resource type, AivenCredentialsActivityLogEntry type, and AIVEN_CREDENTIALS_CREATE activity filter

Follows up on #358 and #362.

@jhrv jhrv requested a review from a team as a code owner March 17, 2026 11:47
Add GraphQL schema extensions for credential creation activity log
entries. Without these, entries were written to the database but would
panic if queried via GraphQL.

Uses generic naming (CREDENTIALS, not AIVEN_CREDENTIALS) since Aiven is
an implementation detail and the types should be reusable for future
credential providers.
@jhrv jhrv force-pushed the fix/aiven-credentials-euthanaisa-label branch from 2bace87 to bc0ed56 Compare March 17, 2026 11:53
Copy link
Contributor

@thokra-nav thokra-nav left a comment

Choose a reason for hiding this comment

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

Lurer litt på om CREDENTIALS er litt for generisk. Men ser ut som at data typen på sikt kommer til å støtte det meste jeg ser for meg som ikke er Aiven også

jhrv added 2 commits March 17, 2026 13:20
Inserts activity log entries via SQL and verifies the GraphQL query
returns them correctly with proper type resolution. This is the test
that would have panicked before adding the schema types.
Remove Aiven from the public-facing GraphQL schema. 'Aiven' is an
implementation detail that end users should not see.
@jhrv jhrv merged commit f298a53 into main Mar 17, 2026
10 checks passed
@jhrv jhrv deleted the fix/aiven-credentials-euthanaisa-label branch March 17, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants