Skip to content

fix(api): include collaborator invite status in notifications#959

Merged
raymondjacobson merged 1 commit into
mainfrom
codex/collab-invite-notification-status
Jun 16, 2026
Merged

fix(api): include collaborator invite status in notifications#959
raymondjacobson merged 1 commit into
mainfrom
codex/collab-invite-notification-status

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

  • enrich track_collaborator_invite notification action data with the current track_collaborators.status at read time
  • document collaborator notification types and optional action status in OpenAPI
  • add a regression test for an invite notification whose collaborator row later becomes accepted

Test Plan

  • go test ./api -run TestV1Notifications -count=1
  • ruby -e "require 'yaml'; YAML.load_file('api/swagger/swagger-v1.yaml'); puts 'swagger yaml ok'"\n- git diff --check\n\n## Notes\nThis makes accepted collaborator invite notifications sticky after reload: the original invite notification row remains immutable, but the API now reads the canonical collaboration row and returns its current status.

@raymondjacobson raymondjacobson merged commit 20d1b91 into main Jun 16, 2026
5 checks passed
@raymondjacobson raymondjacobson deleted the codex/collab-invite-notification-status branch June 16, 2026 22:44
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.

1 participant