Skip to content

request membership part 4 member request interaction backend#1364

Open
fenekku wants to merge 1 commit intoinveniosoftware:masterfrom
fenekku:request_membership_part_4_member_request_interaction_backend
Open

request membership part 4 member request interaction backend#1364
fenekku wants to merge 1 commit intoinveniosoftware:masterfrom
fenekku:request_membership_part_4_member_request_interaction_backend

Conversation

@fenekku
Copy link
Copy Markdown
Contributor

@fenekku fenekku commented Apr 1, 2026

This PR is # 4 in a series of PRs to revive and merge the "Request to join a community as a user" feature. Draft RFC here: inveniosoftware/rfcs#111 .

This PR covers the part of the feature that deals with backend interactions with request memberships. Specifically: actions (accept, decline) on members (membership requests and fix for invitations), links for these actions and role update of membership requests.

Out of bounds/future PRs: any frontend aspect and any notifications.

Current commits

  • feat(mshp-req): accept,decline,cancel,upate role of membership requests

@fenekku fenekku force-pushed the request_membership_part_4_member_request_interaction_backend branch 2 times, most recently from 2c6d31e to 9079eb0 Compare April 20, 2026 17:59
@fenekku fenekku force-pushed the request_membership_part_4_member_request_interaction_backend branch 2 times, most recently from 917717e to 42a3a0b Compare April 30, 2026 14:36
…ts [+]

- refactors invitations to use common methods
@fenekku fenekku force-pushed the request_membership_part_4_member_request_interaction_backend branch from 42a3a0b to e7ef134 Compare April 30, 2026 15:27
Comment on lines +696 to +701
uow.register(
IndexRefreshOp(
indexer=self.indexer,
index=ArchivedMemberRequest.index,
)
)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is just moved from what was in decline_invite. Adding index_refresh=True to RecordCommitOp` above might be equivalent (not too sure why requiring a refresh is really needed - refresh should happen fast enough on its own to not affect UX , no?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needed

Development

Successfully merging this pull request may close these issues.

1 participant