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

Ability for admins to add club ambassador profile to mentors #5254

Conversation

shaun-technovation
Copy link
Contributor

This will let admins add a club ambassador profile to an existing mentor account.

@shaun-technovation shaun-technovation changed the title WIP: Ability for admins to add club ambassador profile to mentors Ability for admins to add club ambassador profile to mentors Dec 18, 2024
Copy link
Contributor

@viviancan viviancan left a comment

Choose a reason for hiding this comment

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

Works as expected! If the mentor is already assigned to a primary chapter should that assignment be removed or at a minimum not be the primary assignment?

@shaun-technovation
Copy link
Contributor Author

If the mentor is already assigned to a primary chapter should that assignment be removed or at a minimum not be the primary assignment?

Good question, this would be an issue for adding a chapter ambassador profile to mentor accounts too.

If we remove the mentor chapter/club assignments then the ambassador for the club/chapter wouldn't be able to see them in their participant's list; which I think they'd want to be able to do if they are mentoring teams in their chapter/club... 🤔

An account can have multiple primary assignments too, a primary assignment for the mentor profile and a primary assignment for the club/chapter ambassador profile wouldn't be a problem (but an individual profile shouldn't have multiple primary assignments).

Anywho, I'm not the one to be making the call on what to do. 😊 @dboyer^?

I'll merge this one in and we can address this in another issue. 👍

@shaun-technovation shaun-technovation force-pushed the 5239-ability-for-admins-to-add-club-ambassador-profile-to-mentor-accounts branch from fc5a4d1 to 30d92e4 Compare December 19, 2024 20:07
@shaun-technovation shaun-technovation merged commit 30d92e4 into preview Dec 19, 2024
1 check was pending
@shaun-technovation shaun-technovation deleted the 5239-ability-for-admins-to-add-club-ambassador-profile-to-mentor-accounts branch December 19, 2024 20:08
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.

2 participants