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

Feature/images #48

Open
wants to merge 35 commits into
base: develop
Choose a base branch
from
Open

Feature/images #48

wants to merge 35 commits into from

Conversation

RohanCruickshank
Copy link
Contributor

No description provided.

 - Update required config settings
 - Create database controller framework
 - Create script for tests
 - Link with commandController
 - Add mongoose types for Typescript
 - Link with timesheet reminder and time reports
 - Update help with commands
 - Add the local image folder to the git-ignore
…the removal of unwanted ones (when removal is implemented)
 - Add help text for !images getall <type> command
 - Update commandController's mock builder to include recently added functions
 - Restructure getImage to allow specific images to be gotten (currently overwrites the current day's image)
 - Add more logging to Discord so that users know that stuff is happening
 - Yet to cover a successful result
 - mock database to allow for interaction testing
…even if there's no database

 - Add checks into databaseController to check that the database is connected before trying to access it
 - Try to optimise tests // For some reason a simple test is taking a full second to complete (Throw an exception)...
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.

1 participant