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

Tents V2 #780

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Tents V2 #780

wants to merge 2 commits into from

Conversation

LunaUrsa
Copy link
Contributor

  • Update tents message, fix for flair overriding

  • Edit flair sizing, add 1 background and 3 fonts

  • Tent rewrite stage 1, added "modes" (locked, hidden, open)

TripBot now uses the database to keep track of tent settings and data.

  • Added persistent ban lists, POC for /voice settings GUI

  • Added whitelisting, simplified lock,hide,open just to private/public

  • Revert .jsons <3

  • Updates and suggestions

  • Fix bugs

  • Voice Menu

  • Disable VC ping while private, add rename tent button modal

  • Added dynamic ID fetching for /voice settings in tent info message

  • Add saved tent name, fancier settings embed, fix for user select menus

  • Added function for clearing and updating permissions, currently only runs on /voice settings ban/whitelist changes

  • Add base minimum join level option

Currently only runs when unlocking or locking tent, requires heavy optimisation and needs a select menu added

  • Min Join Level fully functioning

Added more optimized logic
Added select menu to the settings menu

  • Cleaning up some functions

  • Finishing up join level functionality

  • Remove awaits to prevent timeouts, let slow permission changes happen in the background

  • Fix ping cooldown timestamps

  • We love nested template literals

  • Tip: Don't let copilot lint for you

  • Remove /poll, fix Voice category showing as TOTAL


* Update tents message, fix for flair overriding

* Edit flair sizing, add 1 background and 3 fonts

* Tent rewrite stage 1, added "modes" (locked, hidden, open)

TripBot now uses the database to keep track of tent settings and data.

* Added persistent ban lists, POC for /voice settings GUI

* Added whitelisting, simplified lock,hide,open just to private/public

* Revert .jsons <3

* Updates and suggestions

* Fix bugs

* Voice Menu

* Disable VC ping while private, add rename tent button modal

* Added dynamic ID fetching for /voice settings in tent info message

* Add saved tent name, fancier settings embed, fix for user select menus

* Added function for clearing and updating permissions, currently only runs on /voice settings ban/whitelist changes

* Add base minimum join level option

Currently only runs when unlocking or locking tent, requires heavy optimisation and needs a select menu added

* Min Join Level fully functioning

Added more optimized logic
Added select menu to the settings menu

* Cleaning up some functions

* Finishing up join level functionality

* Remove awaits to prevent timeouts, let slow permission changes happen in the background

* Fix ping cooldown timestamps

* We love nested template literals

* Tip: Don't let copilot lint for you

* Remove /poll, fix Voice category showing as TOTAL

---------

Co-authored-by: Hipperooni <hanzhanrob@gmail.com>
Co-authored-by: Hipperooni <60094069+Hipperooni@users.noreply.github.com>
@LunaUrsa LunaUrsa changed the title Tent moderation overhaul (#779) Tents V2 May 12, 2024
@LunaUrsa
Copy link
Contributor Author

Hey @Hipperooni idk what happened but i think 'tent-moderation-overhaul' was an old branch so it was causing issues on merging with main, so i made this new branch, merged your stuff with this new branch, and we can use this one now.

Couple questions:

  1. We have users.joinTentLevel and then tent_settings.join_level, do we need both?
  2. tent_hostlist.tent and tent_whitelist.tent exist, but then tent_blacklist uses .channel?
  3. users.createdTents is a list, but shouldn't each user have a single tent_settings entry?
    3.1) Shouldn't tent_settings have a unique key so there's only one entry per user?
  4. users.defaultTentName should probably go into the tent_settings?
  5. users.banlist and users.whitelist don't seem to be used anymore and were replaced by the bannedFromTents and whitelistedInTents fields, if so, we should remove those?

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.

None yet

1 participant