Skip to content

[ENG-10943] Add the 'id' and 'event_name' attributes to the 'SubscriptionDetail' updates response#11719

Merged
cslzchen merged 2 commits intoCenterForOpenScience:feature/pbs-26-9from
Ostap-Zherebetskyi:fix/subscription_detail_attr
May 1, 2026
Merged

[ENG-10943] Add the 'id' and 'event_name' attributes to the 'SubscriptionDetail' updates response#11719
cslzchen merged 2 commits intoCenterForOpenScience:feature/pbs-26-9from
Ostap-Zherebetskyi:fix/subscription_detail_attr

Conversation

@Ostap-Zherebetskyi
Copy link
Copy Markdown
Collaborator

@Ostap-Zherebetskyi Ostap-Zherebetskyi commented Apr 30, 2026

Ticket

Purpose

Add the 'id' and 'event_name' attributes to the 'SubscriptionDetail' updates response

Changes

Side Effects

QE Notes

CE Notes

Documentation

@Ostap-Zherebetskyi Ostap-Zherebetskyi changed the title Add the 'id' and 'event_name' attributes to the 'SubscriptionDetail' updates response [ENG-10943] Add the 'id' and 'event_name' attributes to the 'SubscriptionDetail' updates response Apr 30, 2026
@Ostap-Zherebetskyi Ostap-Zherebetskyi marked this pull request as ready for review April 30, 2026 14:03
@Ostap-Zherebetskyi Ostap-Zherebetskyi changed the base branch from hotfix/26.8.1 to develop April 30, 2026 14:27
@Ostap-Zherebetskyi Ostap-Zherebetskyi changed the base branch from develop to feature/pbs-26-9 April 30, 2026 14:27
@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the fix/subscription_detail_attr branch from da7f4c4 to 08a09b3 Compare April 30, 2026 14:28
Copy link
Copy Markdown
Collaborator

@cslzchen cslzchen left a comment

Choose a reason for hiding this comment

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

Looks good overall. In addition to comments, can we update unit tests to make sure id and events are returned in API responses.

Comment thread api/users/views.py
@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the fix/subscription_detail_attr branch from fb94d48 to ad4d546 Compare May 1, 2026 10:47
Copy link
Copy Markdown
Collaborator

@cslzchen cslzchen left a comment

Choose a reason for hiding this comment

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

@cslzchen cslzchen merged commit 0d4bd1d into CenterForOpenScience:feature/pbs-26-9 May 1, 2026
8 checks passed
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