Page MenuHomePhorge

Activate the corresponding room when clicking on notification
ClosedPublic

Authored by tusooa on May 11 2024, 7:13 AM.
Tags
None
Referenced Files
F112622: D24.1732339380.diff
Thu, Nov 21, 9:23 PM
F112563: D24.1732335281.diff
Thu, Nov 21, 8:14 PM
F112459: D24.1732317646.diff
Thu, Nov 21, 3:20 PM
F112150: D24.1732272257.diff
Thu, Nov 21, 2:44 AM
F112149: D24.1732272255.diff
Thu, Nov 21, 2:44 AM
F112148: D24.1732272251.diff
Thu, Nov 21, 2:44 AM
F112147: D24.1732272249.diff
Thu, Nov 21, 2:44 AM
F112146: D24.1732272243.diff
Thu, Nov 21, 2:44 AM
Subscribers

Details

Summary

When the user clicks on a notification, they usually want to see the message right away. This makes kazv lead them to the corresponding room when the notification is clicked on (defaultActivated).

Type: add

BUG: https://iron.lily-is.land/T62

Test Plan

Send a message that creates a notification from another user. Verify that when clicking on the notification, the corresponding room is activated, and the room is always opened as the second page (i.e. index 1) no matter how many pages are opened.

Open kazv and resize the window so that only one page is shown. Open the page of some room. Go back to main page, and send a message from another user to that room. Click on the notification and verify that the room page is shown.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 39
Build 68: GitLab CI for kazv
Build 67: arc lint + arc unit

Event Timeline

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

Harbormaster returned this revision to the author for changes because remote builds failed.May 11 2024, 7:16 AM
Harbormaster failed remote builds in B39: Diff 59!
nannanko subscribed.
nannanko added inline comments.
src/contents/ui/MessageNotification.qml
18

It don't work when sdkVars.currentRoomId was already roomId.

Do you still remember kazv can't return room page with click room item? This made the same mistake.

I fixed the previous one, so I suggested using a function to set the currentRoomId and reuse my code.

This revision now requires changes to proceed.May 29 2024, 5:37 AM

Fix not returning to room in narrow mode

Rebase onto lily/servant

This revision is now accepted and ready to land.Jun 1 2024, 2:54 AM