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

DB Rewrite & Partial refactoring #25

Merged
merged 23 commits into from
Nov 18, 2021
Merged

DB Rewrite & Partial refactoring #25

merged 23 commits into from
Nov 18, 2021

Conversation

fuccsoc
Copy link
Member

@fuccsoc fuccsoc commented Nov 8, 2021

Database & core modules, used that rewrited to FTG-like syntax
Need tests before merge

Install script optimization
- db update
- notes rewrite
- remove old db from utils
@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 8, 2021

  • IT WORKS!
  • Async in db fucked
  • plugins/utils/db.py reviewed
  • Install script reviewed
  • plugins/notes.py tested
  • plugins/sessionkiller.py adapted & tested (thx: @iamnalinor)
  • plugins/AdminTool.py tested (ex: triggers)
  • plugins/AntiPm.py tested

@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 8, 2021

Release is coming...

@fuccsoc fuccsoc marked this pull request as ready for review November 8, 2021 23:37
@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 8, 2021

IT BEGINS!
First tests are finished. Release is very close...

- AntiPM: add settings for sending spam report or not
- Weather: add quick-accessed city
@fuccsoc fuccsoc changed the title [TEST REQUIRED] DB Rewrite DB Rewrite & Partial refactoring Nov 9, 2021
@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 9, 2021

Found bag: .tunmute @user doesn't make tunmute

@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 9, 2021

@fuccsoc fixed

@fuccsoc
Copy link
Member Author

fuccsoc commented Nov 16, 2021

Tested and ready to merge.

@john-phonk john-phonk merged commit bfeba8e into Dragon-Userbot:master Nov 18, 2021
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.

None yet

3 participants