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.

Sign up for GitHub

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

Jump to bottom

fix(CommunitySettings): Process statuses for members model #6341

Merged
merged 1 commit into from
Jul 8, 2022
Merged

fix(CommunitySettings): Process statuses for members model #6341

merged 1 commit into from
Jul 8, 2022

Conversation

borismelnik
Copy link
Contributor

Closes: #6132

What does the PR do

Process statuses for members model

Affected areas

Community overview

Screenshot of functionality

Снимок экрана 2022-07-04 в 16 55 15

@borismelnik borismelnik requested review from a team, Hbouaz, iurimatias and caybro and removed request for a team July 4, 2022 14:00
@status-im-auto
Copy link
Member

status-im-auto commented Jul 4, 2022 •
edited
Loading

Jenkins Builds

Click to see older builds (4)
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
⁉️ 4be688c #1 2022-07-04 14:03:37 ~2 min linux-cpp 📄log
✔️ 4be688c #1 2022-07-04 14:09:06 ~8 min macos 📦dmg
✔️ 4be688c #1 2022-07-04 14:12:19 ~11 min linux 📦tgz
✔️ 4be688c #1 2022-07-04 14:30:31 ~29 min windows 📦exe
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
⁉️ c948a22 #2 2022-07-04 17:00:00 ~2 min linux-cpp 📄log
✔️ c948a22 #2 2022-07-04 17:07:51 ~10 min linux 📦tgz
✔️ c948a22 #2 2022-07-04 17:14:29 ~16 min macos 📦dmg
✔️ c948a22 #2 2022-07-04 17:20:53 ~23 min windows 📦exe
⁉️ 16e269d #3 2022-07-06 17:28:54 ~2 min linux-cpp 📄log
✔️ 16e269d #3 2022-07-06 17:34:34 ~7 min macos 📦dmg
✔️ 16e269d #3 2022-07-06 17:38:56 ~12 min linux 📦tgz
✔️ 16e269d #3 2022-07-06 17:47:27 ~20 min windows 📦exe

@borismelnik borismelnik requested a review from osmaczko July 4, 2022 16:57
jrainville
jrainville reviewed Jul 4, 2022
View reviewed changes
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.

I feel like some of the changes here are the same as @endulab 's PR #6058

@endulab
Copy link
Contributor

endulab commented Jul 5, 2022

I feel like some of the changes here are the same as @endulab 's PR #6058

Yes, A part of the pr is already included in #6058. I am going to merge #6058 today, so this pr will have to be slightly updated.

@caybro caybro removed their request for review July 5, 2022 12:19
@Hbouaz Hbouaz added the tested label Jul 6, 2022
@borismelnik borismelnik merged commit 81e674d into master Jul 8, 2022
@borismelnik borismelnik deleted the fix/issue-6132-fix-statuses branch July 8, 2022 09:58
This was referenced Jul 8, 2022
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrainville jrainville jrainville approved these changes

@Hbouaz Hbouaz Hbouaz approved these changes

@osmaczko osmaczko osmaczko approved these changes

@iurimatias iurimatias Awaiting requested review from iurimatias iurimatias was automatically assigned from status-im/desktop-devs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Community Settings Member section not display statuses
6 participants
@borismelnik @status-im-auto @endulab @jrainville @Hbouaz @osmaczko