# Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
- Fix bug where a default config setting wouldn't show the key you were using
- Fix bug where a config which doesn't exist wouldn't show the key you were using
#289
## Type of change
Please delete options that are not relevant.
- [x] Bug fix (non-breaking change which fixes an issue)
# How Has This Been Tested?
Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.
- This was tested by checking both a setting which is defaulted and not used, and both now show they key correctly
# Checklist
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules
Reviewed-on: https://gitea.vylpes.xyz/RabbitLabs/vylbot-app/pulls/388
Reviewed-by: VylpesTester <tester@vylpes.com>
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>
# Description
- Fix the invalid config value not showing properly
- When there is no server value or default value, it will show it as `<NONE>`
#315
## Type of change
Please delete options that are not relevant.
- [x] Bug fix (non-breaking change which fixes an issue)
# How Has This Been Tested?
Please describe the tests that you ran to verify the changes. Provide instructions so we can reproduce. Please also list any relevant details to your test configuration.
# Checklist
- [x] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that provde my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [x] Any dependent changes have been merged and published in downstream modules
Reviewed-on: https://gitea.vylpes.xyz/RabbitLabs/vylbot-app/pulls/336
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>
# Description
- Fixed the `/config get` command not returning the actual key's name, instead using a strinified "Object" property
#289
## Type of change
Please delete options that are not relevant.
- [x] Bug fix (non-breaking change which fixes an issue)
# How Has This Been Tested?
- This was tested by running the `/config get` command on my dev bot, and making sure it still gets the correct value for the key string
# Checklist
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that provide my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [x] Any dependent changes have been merged and published in downstream modules
Reviewed-on: https://gitea.vylpes.xyz/RabbitLabs/vylbot-app/pulls/334
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>
- Switch to TypeORM's DataSource API, rather than using the now deprecated ormconfig.json
- This will fix stage deployment not knowing how to deploy the database migrations
#297
> **NOTE:** This change requires the deployment scripts to be updated, please update them on the server before merging
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Reviewed-on: https://gitea.vylpes.xyz/RabbitLabs/vylbot-app/pulls/299
* Change rules.txt to rules.json (#31)
* Migrate to yarn
* Add role configs to config template
* Install packges and setup typescript
* Migrate entry point
* Migrate about command
* Migrate ban command
* Migrate clear command
* Migrate kick command
* Migrate mute command
* Migrate poll command
* Migrate bunny command
* Update required roles checker
* Migrate role command
* Migrate unmute command
* Migrate warn command
* Migrate eval command
* Migrate help command
* Migrate rules command
* Migrate events to typescript
* Update about command to use the PublicEmbed class
* Update ErrorMessage to ChannelNotFound
* Update messageDelete event to ignore bots
* Feature/74 merge vylbot core (#80)
* Merge VylBot-Core
* Update commands to new system
* Fix issue where events would not load
* Feature/12 create tests (#102)
* Fix tests
* Update coverage
* Remove unrequired mock files
* Add about command test
* Update about tests
* Ban command tests
* eval command tests
* Start help command tests
* Add help command tests
* Add kick command tests
* Mute command tests
* Poll command tests
* Add role command tests
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Add rules command tests
* Add unmute command tests
* Add warn command tests
* Add MemberEvents tests
* Add GuildMemberUpdate tests
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Add MessageEvents tests
* Add StringTools test
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Add embed tests
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Add GitHub Actions
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Move to tslint
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Remove tslint
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Remove linting script
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Update rules with blog website and event spoilers rule" (#106)
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Containerise bot (#107)
* Add moderator names to audit reason (#108)
* Feature/48 database (#114)
* Add database and default values
* Add ability to save a setting to the database
* Get commands and events to use database
* Setup and config command
* Update commands to check roles per server
* Different rules per server
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Different prefix per server
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Add verification system
Signed-off-by: Ethan Lane <ethan@vylpes.com>
* Disabled commands per server
* Add devmode for default prefix
* Update embeds
* Fix broken tests
* Feature/66 add different commands per server (#122)
* Add ability for server exclusive commands
* Add MankBot server-exclusive commands
* Add lobby entity to database
* Add documentation
* Add setup command for lobby (#123)
* Update bot to discord.js v13 (#125)
* Update bot to discord.js v13
* Remove debug code
* 110 commandshelp about command errors which causes command to not run (#126)
* Change onMessage to onMessageCreate
* Fix help command
* Add override for bot owner and server owner (#135)
* Change help command so exclusive commands can only be seen for the server they're assigned to (#136)
* Change parsing to not crash if invalid (#142)
* 137 role command cannot read properties of undefined (#141)
* Fix issue with bot crashing
* Fix server prefix not showing
* Add easy way to configure role command
* Move help text to its own directory
* Make role config command to use role id
* Get lobby command to use IDs instead of names (#144)
Co-authored-by: Vylpes <getgravitysoftware@gmail.com>
