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

feat(hub): allow some roles to create new hubs #3126

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

wescopeland
Copy link
Member

Depends on #3065 and #3084.

Allows admins, game hash managers, and game editors to create new hubs.

Screenshot 2025-01-24 at 9 36 52 PM

Screenshot 2025-01-24 at 9 36 58 PM

@wescopeland wescopeland requested a review from a team January 25, 2025 02:38
Role::ADMINISTRATOR,
Role::GAME_HASH_MANAGER,
Role::GAME_EDITOR,
]);
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if we should have a separate role, or if this should be opened more.

I know we have hubs dedicated to events, and there's all the Meta|** hubs that are used by other teams.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not too sure either. I don't think this should be opened up to events managers or the 3 staff dev roles. Right now I'm inclined to think if any of those folks regularly find themselves creating new hubs, they should be given the game editor role, otherwise they can ask one of these 3 roles who are ultimately responsible for the curation of the hubs.

@wescopeland wescopeland merged commit fda04f1 into RetroAchievements:master Jan 28, 2025
8 checks passed
@wescopeland wescopeland deleted the hub-creation branch January 28, 2025 21:59
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