zio/bsky-app
2
0
Fork 0
Code Pull requests Releases Packages Activity

Fix: avoid duplicate key names in notifications

Browse source
This commit is contained in:
Paul Frazee 2022-12-13 15:35:11 -06:00
parent 8af8d782c7
commit ac97667206
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/state/models/notifications-view.ts
View file

@ -11,6 +11,8 @@ const UNGROUPABLE_REASONS = ['trend', 'assertion']
const PAGE_SIZE = 30
const MS_60MIN = 1e3 * 60 * 60
let _idCounter = 0
export interface GroupedNotification extends ListNotifications.Notification {
additional?: ListNotifications.Notification[]
}
@ -361,13 +363,12 @@ export class NotificationsViewModel {
private async _appendAll(res: ListNotifications.Response, replace = false) {
this.loadMoreCursor = res.data.cursor
this.hasMore = !!this.loadMoreCursor
let counter = this.notifications.length
const promises = []
const itemModels: NotificationsViewItemModel[] = []
for (const item of groupNotifications(res.data.notifications)) {
const itemModel = new NotificationsViewItemModel(
this.rootStore,
`item-${counter++}`,
`item-${_idCounter++}`,
item,
)
if (itemModel.needsAdditionalData) {