-
Notifications
You must be signed in to change notification settings - Fork 1
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
Major version update | 2.2.1 --> 2.3.0 #44
Draft
nathen418
wants to merge
177
commits into
main
Choose a base branch
from
staging
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Move utility functions and modules into utils folder # Changes * Create utils folder * Move `utils.ts` to utils folder * Rename `roleOps.ts` to `roleUtils.ts` and move to utils folder
…into feature/nathen418/detect-containerization
…ithub.com/Antares-Network/CSSC-Bot into feature/nathen418/detect-containerization
Save course id to db
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.8.2 to 4.8.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Commits](microsoft/TypeScript@v4.8.2...v4.8.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
# 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
# Deepsource fix replace == with ===
# Sort channels
# Compare db roles against server roles instead of channels
# Channels id's were getting printed instead of name
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Bug Fix
dependencies
Pull requests that update a dependency file
New Features
Requires Careful Review
Version 2.2.x
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New in Version 2.2.2
/cscreatechannels
command to create cs channels for every entry in theclass
database and ping all the people in that class that a new channel was created.class
database to contain Channel ID's and Channel Names.Merged Pull Requests
Full Changelog: v2.2.1...v2.2.2