Skip to content

Commit

Permalink
[Glitch] Fix issue with grouped notifications UI due to recent API ch…
Browse files Browse the repository at this point in the history
…ange

Port a50c8e9 to glitch-soc

Signed-off-by: Claire <[email protected]>
  • Loading branch information
ClearlyClaire committed Jul 31, 2024
1 parent de8a31f commit 49ad147
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app/javascript/flavours/glitch/api_types/notifications.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ export interface BaseNotificationGroupJSON {

interface NotificationGroupWithStatusJSON extends BaseNotificationGroupJSON {
type: NotificationWithStatusType;
status: ApiStatusJSON;
status_id: string;
}

interface NotificationWithStatusJSON extends BaseNotificationJSON {
Expand Down
4 changes: 2 additions & 2 deletions app/javascript/flavours/glitch/models/notification_group.ts
Original file line number Diff line number Diff line change
Expand Up @@ -124,9 +124,9 @@ export function createNotificationGroupFromJSON(
case 'mention':
case 'poll':
case 'update': {
const { status, ...groupWithoutStatus } = group;
const { status_id: statusId, ...groupWithoutStatus } = group;
return {
statusId: status.id,
statusId,
sampleAccountIds,
...groupWithoutStatus,
};
Expand Down

0 comments on commit 49ad147

Please sign in to comment.