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.

Sign up for GitHub

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

Jump to bottom

Put TITLE in duplicate channel and role names #50

Open
schiltz3 opened this issue Sep 16, 2022 · 0 comments · Fixed by #48
Open

Put TITLE in duplicate channel and role names #50

schiltz3 opened this issue Sep 16, 2022 · 0 comments · Fixed by #48
Assignees
@schiltz3
Labels
Feature Request

Comments

@schiltz3
Copy link
Collaborator

schiltz3 commented Sep 16, 2022 •
edited
Loading

Instead of using <name>-0, <name>-1 to differentiate between duplicate names, use <name>-<title1> and <name>-<title2>
@schiltz3 schiltz3 mentioned this issue Sep 16, 2022
Merged
@schiltz3 schiltz3 linked a pull request Sep 16, 2022 that will close this issue
Feature/schiltz3/db migrate #48
Merged
@schiltz3 schiltz3 self-assigned this Sep 16, 2022
@schiltz3 schiltz3 moved this to In Progress in CSSC-Bot Project Board Sep 16, 2022
schiltz3 added a commit that referenced this issue Jan 18, 2023
@schiltz3
Feature/schiltz3/db migrate (#48)
f3a0597 
# Migrate database to clean up structure & rewrite database calls to
match

Edit the structure

* CODE => NAME
* TITLE
* INFO
* ~~ROLE_NAME~~
* ~~CHANNEL_NAME~~
* ROLE_ID
* CHANNEL_ID
* ~~UUID~~
* \+ DUPE
* \+ ACTIVE

This amounts to:
* Removing `UUID`, `ROLE_NAME` and `CHANNEL_NAME`.
* Cleaning up `CODE` and replacing it with `NAME`.
* Adding the `DUPE` field to signify if a class is a duplicate.
* Adding `ACTIVE` to specify if the course is being offered that
semester. (Currently, all classes are considered active.)

### Steps to migrate database and discord server
Run `migrateDb` command to migrate the database
Run `CreateChannels` command to update channel names from the database
Run `CreateRoles` command to update role names from the database

# Link to github cards
#33 
#35 
#49 
#50

# Additions

* Add a helper function to create embeds
* Add `migrateDB` command to update the database to the new schema,
update how duplicate codes are handled, and clean all strings.
* Add modified JS Docs to function. (Types are not added since this is
specified in function definitions with typescript)

# Changes

* Change db scheme
* Change role and channel db calls to reflect changes
* Change how duplicate classes are handled
* Use course title and info in channel topics



## Bugfix
* replace all contiguous whitespace with a dash instead of replacing
spaces with a dash
* Replace contiguous dashes with a dash in ROLE_NAME and CHANNEL_NAME
* Trim leading and trailing whitespace from TITLE
* Limit role name and channel name to 100 characters each
* Limit channel topic to 1024 characters each
* Only reply to role selection if role is actually assigned
* Add bottleneck package to limit the bot to 50 API calls per second
* Prefers matching on channel ID's rather than names
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schiltz3 schiltz3

Labels
Feature Request
Projects
CSSC-Bot Project Board
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/schiltz3/db migrate Antares-Network/CSSC-Bot
1 participant
@schiltz3