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(Communities): Add ephemerial notification when request to join was accepted #7264

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

borismelnik
Copy link
Contributor

Part of: #7072

What does the PR do

Add ephemerial notification when request to join was accepted

Affected areas

Communities

Screenshot of functionality (including design for comparison)

  • I've checked the design and this PR matches it

Снимок экрана 2022-09-07 в 17 46 45

@borismelnik borismelnik requested review from a team, elina2015, iurimatias and endulab and removed request for a team September 7, 2022 14:50
@status-im-auto
Copy link
Member

status-im-auto commented Sep 7, 2022

Jenkins Builds

Click to see older builds (11)
Commit #️⃣ Finished (UTC) Duration Platform Result
⁉️ 10b5d1b #1 2022-09-07 14:56:24 ~5 min linux-cpp 📄log
✔️ 10b5d1b #1 2022-09-07 15:03:26 ~12 min linux 📦tgz
✔️ 10b5d1b #1 2022-09-07 15:16:39 ~25 min windows 📦exe
✔️ 10b5d1b #1 2022-09-07 15:30:44 ~39 min macos 📦dmg
✔️ 0c5dc0b #2 2022-10-03 10:57:41 ~13 min linux-cpp 📦pkg
✔️ 0c5dc0b #2 2022-10-03 11:06:15 ~21 min linux 📦tgz
✔️ 0c5dc0b #2 2022-10-03 11:09:26 ~25 min windows 📦exe
0c5dc0b #2 2022-10-03 11:41:36 ~10 min e2e 📄log
0c5dc0b #2 2022-10-03 12:00:51 ~1 hr 16 min macos 📄log
✔️ 0c5dc0b #3 2022-10-12 14:56:36 ~10 min macos 📦dmg
⁉️ 0c5dc0b #3 2022-10-12 16:00:18 ~10 min e2e 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ bf20af9 #3 2022-10-21 11:40:56 ~4 min linux-cpp 📦pkg
✔️ bf20af9 #4 2022-10-21 11:45:44 ~9 min macos 📦dmg
✔️ bf20af9 #3 2022-10-21 11:48:21 ~12 min linux 📦tgz
bf20af9 #4 2022-10-21 11:58:19 ~12 min e2e 📄log
✔️ bf20af9 #3 2022-10-21 11:59:57 ~23 min windows 📦exe
✔️ 624623c #4 2022-10-27 18:10:34 ~4 min linux-cpp 📦pkg
✔️ 624623c #5 2022-10-27 18:15:08 ~9 min macos 📦dmg
✔️ 624623c #4 2022-10-27 18:17:02 ~11 min linux 📦tgz
✔️ 624623c #4 2022-10-27 18:29:48 ~23 min windows 📦exe
⁉️ 624623c #5 2022-10-27 18:45:26 ~24 min e2e 📄log

@@ -696,6 +696,7 @@ method communityJoined*[T](
community.joined):
singletonInstance.globalEvents.myRequestToJoinCommunityAcccepted("Community Request Accepted",
fmt "Your request to join community {community.name} is accepted", community.id)
self.displayEphemeralNotification(fmt "{community.name} membership approved ", "", conf.COMMUNITIESPORTAL_SECTION_ICON, false, EphemeralNotificationType.Success.int, "")
Copy link
Member

Choose a reason for hiding this comment

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

and same here :) Need to move this to QML

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.

Please create the issue for the notifications to be translatable :)

@borismelnik borismelnik force-pushed the feat/issue-7072-accept-request-notification branch from 10b5d1b to 0c5dc0b Compare October 3, 2022 10:43
@borismelnik
Copy link
Contributor Author

Please create the issue for the notifications to be translatable :)

@caybro #8002

@borismelnik borismelnik force-pushed the feat/issue-7072-accept-request-notification branch from 0c5dc0b to bf20af9 Compare October 21, 2022 11:35
@borismelnik borismelnik force-pushed the feat/issue-7072-accept-request-notification branch from bf20af9 to 624623c Compare October 27, 2022 18:05
@borismelnik borismelnik merged commit 6de3466 into master Oct 28, 2022
@borismelnik borismelnik deleted the feat/issue-7072-accept-request-notification branch October 28, 2022 08:02
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.

None yet

5 participants