Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

v2.0.0 #158

Merged
merged 202 commits into from
Jan 24, 2019
Merged

v2.0.0 #158

merged 202 commits into from
Jan 24, 2019

Conversation

Galarzaa90
Copy link
Owner

No description provided.

- Added new dependencies to requirements.txt
- NabBot is now started from launcher.py
- Fixed bug in NabBot startup function
- Skip duplicates when migrating levelups and deaths
- /sql command now uses postgresql
- Minor complexity improvements
- Temporarily disabled tracking tasks.
- get_character() calls need to be updated as the signature changed.
- Level up's date default value is now()
- Added UNIQUE constraint to character_id and date (a character can't have two deaths at the exact same time)
- Fixed prefix migration
- Changed time zone types in database to use time zone
- Fixed sql query error in get_character's guild updating
- Prefixes are now obtained from postgresql
- Command statistics are now saved
- Fixed command stats timezone
- /levels, /deaths and /timeline are no longer available on PMs
- Fixed bug with the prefix callable
- Fixed types for some event columns
- Split import functions into smaller functions
- Removed figureAltCaption from requirements-docs.txt as it is no longer compatible
- Renamed some columns of event table
- Reenabled highscores for get_character()
# Conflicts:
#	cogs/tibia.py
#	requirements-docs.txt
- Moved legacy sqlite code to database_migration
- Fixed travis job
Replace sqlite database with postgresql
- I forgot to change record references to use tuples instead of the previously used row factory.
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21720 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21724 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21730 lines exceeds the maximum allowed for the inline comments feature.

- Organized changelog, preferred order is feature, improvement, bug
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21767 lines exceeds the maximum allowed for the inline comments feature.

- Fixed error when logs folder doesn't exist
- Documented some functions
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21765 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21777 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 21777 lines exceeds the maximum allowed for the inline comments feature.

@codeclimate
Copy link

codeclimate bot commented Jan 24, 2019

Code Climate has analyzed commit 3a3b7c6 and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 8

Note: there are 3 critical issues.

View more on Code Climate.

@Galarzaa90 Galarzaa90 merged commit d6296fb into master Jan 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants