Skip to content

Combine Functionality for Members and Private Participants in Assemblies#13502

Merged
alecslupu merged 43 commits intodevelopfrom
feat/merge-members-and-private-participants
Dec 12, 2024
Merged

Combine Functionality for Members and Private Participants in Assemblies#13502
alecslupu merged 43 commits intodevelopfrom
feat/merge-members-and-private-participants

Conversation

@mllocs
Copy link
Copy Markdown
Contributor

@mllocs mllocs commented Oct 7, 2024

🎩 What? Why?

Is your feature request related to a problem? Please describe. Currently, there are two functionalities that overlap in some aspects and can be confusing: Private participants and members. This is because most of the time, members and private participants coincide. It is not useful to have the list of members unlinked to the access permissions for that private space; often this means having to upload the same list of participants twice.

📌 Related Issues

Fixes #13414

Testing

📷 Screenshots

Screenshot 2024-10-15 at 09 05 01 Screenshot 2024-10-15 at 09 11 04 Screenshot 2024-10-15 at 09 11 17 Screenshot 2024-10-15 at 09 11 32

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']

@mllocs mllocs force-pushed the feat/merge-members-and-private-participants branch from 4de0889 to 5c8023e Compare October 8, 2024 07:23
@alecslupu alecslupu added type: feature project: 2024-developments Barcelona City Council contract labels Oct 8, 2024
github-actions[bot]
github-actions bot previously approved these changes Oct 8, 2024
github-actions[bot]
github-actions bot previously approved these changes Oct 8, 2024
github-actions[bot]
github-actions bot previously approved these changes Oct 10, 2024
@mllocs mllocs self-assigned this Oct 10, 2024
@mllocs
Copy link
Copy Markdown
Contributor Author

mllocs commented Oct 10, 2024

⚠️ @andreslucena @alecslupu @entantoencuanto Looks like there are lots of tests failing in develop. Some are related to imagemagick but other don't, for example: rspec './spec/system/admin/admin_manages_assembly_admins_spec.rb[1:1:2]' which is related to custom domains.

@mllocs mllocs force-pushed the feat/merge-members-and-private-participants branch from bdaa756 to f195302 Compare October 11, 2024 11:03
github-actions[bot]
github-actions bot previously approved these changes Oct 11, 2024
github-actions[bot]
github-actions bot previously approved these changes Oct 11, 2024
github-actions[bot]
github-actions bot previously approved these changes Oct 11, 2024
github-actions[bot]
github-actions bot previously approved these changes Dec 2, 2024
@mllocs mllocs requested a review from alecslupu December 9, 2024 10:14
@alecslupu
Copy link
Copy Markdown
Contributor

alecslupu commented Dec 10, 2024

@mllocs Can you fix the failing pipelines? Also, sync with latest develop to make sure we do not have pipeline issues after merging :)

@mllocs
Copy link
Copy Markdown
Contributor Author

mllocs commented Dec 11, 2024

@mllocs Can you fix the failing pipelines? Also, sync with latest develop to make sure we do not have pipeline issues after merging :)

@alecslupu Done!

Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

@alecslupu alecslupu merged commit 21cd727 into develop Dec 12, 2024
@alecslupu alecslupu deleted the feat/merge-members-and-private-participants branch December 12, 2024 10:45
@furilo furilo requested a review from NilHomedes December 12, 2024 10:45
@furilo
Copy link
Copy Markdown
Contributor

furilo commented Dec 12, 2024

@NilHomedes nevermind the review request

antopalidi pushed a commit that referenced this pull request Dec 16, 2024
…ies (#13502)

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>
@andreslucena andreslucena added the type: change PRs that implement a change for an existing feature label Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Combine Functionality for Members and Private Participants in Assemblies

6 participants