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

[#855] 5) wait for decision flow #1171

Closed

Conversation

fenekku
Copy link
Contributor

@fenekku fenekku commented Jul 5, 2024

Part of #855

Implement the "waiting flow". Consists of 4 things:

  • adapt community header after request from requester's perspective

membership_discussion_link_records_page

membership_discussion_link

  • show membership requests in community from reviewer's perspective
    membership_requests_in_community_page

  • show membership requests in dashboard from requester's side
    requests_by_user_page

  • don't show membership requests in normal community requests from reviewer's perspective
    requests_for_comunity_page

Side-bonus: serialize links for member API correctly. This allows the cancellation of Invitations from the listing directly:

image

After this is merged and released: this inveniosoftware/invenio-app-rdm#2757 in invenio-app-rdm should be looked at (invenio-app-rdm one depends on this current one in invenio-communities)

This PR

Previous PRs

@fenekku fenekku changed the title 855 5 wait for decision flow [#855] 5) wait for decision flow Jul 5, 2024
@fenekku fenekku force-pushed the 855_5_wait_for_decision_flow branch from 443941e to 6dcc6ae Compare July 11, 2024 17:23
@fenekku fenekku force-pushed the 855_5_wait_for_decision_flow branch from e845d6b to 17f1e00 Compare July 19, 2024 19:31
…tend [+]

This concludes the 2nd flow of the membership request feature.
Remaining flows are
- 'waiting for decision' flow
- 'making a decision' flow

This PR needs to be complemented by:
- one in invenio-requests (done)
- one in invenio-rdm-records (to do)
…tend [+]

This concludes the 2nd flow of the membership request feature.
Remaining flows are
- 'waiting for decision' flow
- 'making a decision' flow

This PR needs to be complemented by:
- one in invenio-requests (done)
- one in invenio-rdm-records (done)
@fenekku fenekku force-pushed the 855_5_wait_for_decision_flow branch from 17f1e00 to 67a28b4 Compare July 22, 2024 12:50
@fenekku
Copy link
Contributor Author

fenekku commented Jul 30, 2024

Close in favor of #1175 which contains it.

@fenekku fenekku closed this Jul 30, 2024
@fenekku fenekku deleted the 855_5_wait_for_decision_flow branch July 30, 2024 12:46
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.

1 participant