Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ActivityCenter): Fetch notifications on scroll to bottom #12712

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

MishkaRogachev
Copy link
Contributor

@MishkaRogachev MishkaRogachev commented Nov 13, 2023

Close #9637
Partially fixes #12646

What does the PR do

  • When new notifications arrive, respect timestamp when inserting notifications to the model
  • When AC notifications list is scrolled to bottom, fetch a new portion of AC notifications with the cursor
  • Fix available AC groups on fetching & new notifications arrive
  • Fix a few qml warnings

Affected areas

Activity Center

Video of functionality

Screen.Recording.2023-11-13.at.18.38.31.mov

(on video limit of notifications is 7, default is 20)

@status-im-auto
Copy link
Member

status-im-auto commented Nov 13, 2023

Jenkins Builds

Click to see older builds (15)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 5b498fd #1 2023-11-13 14:52:39 ~6 min tests/nim 📄log
✔️ 5b498fd #1 2023-11-13 14:54:59 ~8 min macos/aarch64 🍎dmg
✔️ 5b498fd #1 2023-11-13 14:56:59 ~10 min macos/x86_64 🍎dmg
✔️ 5b498fd #1 2023-11-13 14:58:35 ~11 min tests/ui 📄log
✔️ 5b498fd #1 2023-11-13 15:01:19 ~14 min linux/x86_64 📦tgz
✖️ 5b498fd #1 2023-11-13 15:20:56 ~34 min tests/e2e 📄log
✔️ 5b498fd #1 2023-11-13 15:22:12 ~35 min windows/x86_64 💿exe
✔️ dc51178 #3 2023-11-14 10:44:12 ~7 min macos/aarch64 🍎dmg
✔️ dc51178 #3 2023-11-14 10:45:08 ~8 min tests/nim 📄log
✔️ dc51178 #3 2023-11-14 10:49:32 ~13 min macos/x86_64 🍎dmg
✔️ dc51178 #3 2023-11-14 10:55:31 ~19 min linux/x86_64 📦tgz
✔️ dc51178 #3 2023-11-14 10:56:03 ~19 min tests/ui 📄log
✖️ dc51178 #3 2023-11-14 10:58:04 ~21 min tests/e2e 📄log
✔️ dc51178 #3 2023-11-14 11:06:23 ~30 min windows/x86_64 💿exe
✔️ dc51178 #4 2023-11-14 12:31:06 ~38 min tests/e2e 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 0adab88 #4 2023-11-14 14:57:00 ~6 min tests/nim 📄log
✔️ 0adab88 #4 2023-11-14 14:59:27 ~8 min macos/x86_64 🍎dmg
✔️ 0adab88 #4 2023-11-14 15:01:51 ~11 min tests/ui 📄log
✔️ 0adab88 #4 2023-11-14 15:08:03 ~17 min linux/x86_64 📦tgz
✔️ 0adab88 #4 2023-11-14 15:19:38 ~28 min windows/x86_64 💿exe
✔️ 0adab88 #5 2023-11-14 15:25:38 ~34 min tests/e2e 📄log
✔️ 967a901 #5 2023-11-14 21:06:16 ~6 min tests/nim 📄log
✔️ 967a901 #5 2023-11-14 21:10:45 ~10 min macos/x86_64 🍎dmg
✔️ 967a901 #5 2023-11-14 21:12:06 ~11 min tests/ui 📄log
✔️ 967a901 #5 2023-11-14 21:18:57 ~18 min linux/x86_64 📦tgz
✔️ 967a901 #6 2023-11-14 21:20:27 ~20 min macos/aarch64 🍎dmg
✔️ 967a901 #6 2023-11-14 21:32:42 ~32 min tests/e2e 📄log
✔️ 967a901 #5 2023-11-14 21:38:30 ~38 min windows/x86_64 💿exe

@MishkaRogachev
Copy link
Contributor Author

@jrainville the ui is little bit laggy, i can make fetch function async for smoother UX

Copy link
Member

@jrainville jrainville 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, but like you said, using the async version of the call would be better

@MishkaRogachev MishkaRogachev force-pushed the feat/activity-center-fetching-with-cursor branch from 5b498fd to 4db2a2a Compare November 14, 2023 10:26
@MishkaRogachev MishkaRogachev force-pushed the feat/activity-center-fetching-with-cursor branch from 4db2a2a to dc51178 Compare November 14, 2023 10:36
Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

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

LGTM

@MishkaRogachev MishkaRogachev force-pushed the feat/activity-center-fetching-with-cursor branch from dc51178 to 0adab88 Compare November 14, 2023 14:50
@MishkaRogachev MishkaRogachev force-pushed the feat/activity-center-fetching-with-cursor branch from 0adab88 to 967a901 Compare November 14, 2023 20:59
@MishkaRogachev MishkaRogachev merged commit 53d19b0 into master Nov 15, 2023
8 of 9 checks passed
@MishkaRogachev MishkaRogachev deleted the feat/activity-center-fetching-with-cursor branch November 15, 2023 11:29
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.

Sorting of Activity Center notification items is wrong Support fetching new notifications on scrolling
4 participants