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

IWB-1: added useBlacklist hook #289

Merged
merged 38 commits into from
Aug 15, 2024

Conversation

FrankreedX
Copy link
Contributor

@FrankreedX FrankreedX commented Jul 14, 2024

IWB stands for Invite, Whitelist, and Blacklist

@FrankreedX FrankreedX changed the title InviteWaitlistBlacklist-1: added useBlacklist hook IWB-1: added useBlacklist hook Jul 16, 2024
Copy link
Contributor

@Gehrkej Gehrkej left a comment

Choose a reason for hiding this comment

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

Aside from the stray comment and ban functionality not working, The useBlacklist hook seems to be working. Lets be sure to get #299 merged in before this PR.

app/content/team/users/[user]/index.js Outdated Show resolved Hide resolved
dbOperations/hooks/useBlackList.js Outdated Show resolved Hide resolved
dbOperations/removeUser.js Outdated Show resolved Hide resolved
@FrankreedX FrankreedX requested a review from Gehrkej August 7, 2024 01:24
…foreignManagement1_blacklist_hook

# Conflicts:
#	app/content/team/users/[user]/index.js
@FrankreedX FrankreedX changed the base branch from main to frankreed/remove_user_invalid_argument August 7, 2024 06:57
Copy link
Contributor

@solderq35 solderq35 left a comment

Choose a reason for hiding this comment

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

LGTM

Successfully banned a user from coach account. Signed into the banned user's account and confirmed I got the ban message.

image

Base automatically changed from frankreed/remove_user_invalid_argument to main August 12, 2024 14:36
@FrankreedX FrankreedX force-pushed the frankreed/foreignManagement1_blacklist_hook branch from fca54a9 to 310b3f9 Compare August 13, 2024 06:07
@FrankreedX FrankreedX changed the base branch from main to frankreed/fix_verified_redirect_bug August 13, 2024 11:23
Base automatically changed from frankreed/fix_verified_redirect_bug to main August 15, 2024 13:00
@FrankreedX FrankreedX merged commit 76fb5c7 into main Aug 15, 2024
1 check passed
@FrankreedX FrankreedX deleted the frankreed/foreignManagement1_blacklist_hook branch August 15, 2024 13:00
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.

3 participants