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

#8689zbgd9 Memberships: registry #367

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

sab-LC
Copy link
Collaborator

@sab-LC sab-LC commented Oct 14, 2024

Issue:

Description:
Currently in the registry, communities only appear if they have been confirmed, or when the is_approved flag is set to True .

This will need to be changed to show all created communities on the hub, but communities that are members will have an orange checkmark next to their names, like with subscriptions.

Communities that are NOT members (will need to confirm this):
When a user goes to the public account page of that community, they will be able to contact the community but not Request to join , this option will be disabled.

Solution:

  • Updated the query to fetch on member communities to show on registry
  • Updated the _community-card template to include the orange checkmark next to member communities name
  • Updated the public template to include orange checkmark and hide request to join button

Before:

Memebr-registry.before.mp4

After:

Member-registry.after.mp4

@sab-LC sab-LC self-assigned this Oct 14, 2024
@arojas1
Copy link
Contributor

arojas1 commented Oct 14, 2024

Task linked: CU-8689zbgd9 Memberships: registry

@sab-LC sab-LC marked this pull request as ready for review October 14, 2024 14:51
Base automatically changed from 8689zbg4q-community-account-creation to subscriptions October 15, 2024 21:08
@birbjam birbjam merged commit 919a340 into subscriptions Oct 15, 2024
1 check passed
@birbjam birbjam deleted the 8689zbgd9-membership-registry branch October 15, 2024 21:09
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.

4 participants