Gacha_response modification #11

Open
waifu wants to merge 2 commits from dev into master
waifu commented 1 day ago
Owner

Referencing issue #3

  • Modified the bot_app.py process
    • Notifications don't get sent to the parser if they aren't a mention
    • note variables more descriptive
    • Response gets sent directly from bot_app.py
    • Visibility gets checked here too instead of going inside of the function
    • Sleep time lowered to two so testing is faster
      • There may be a way to mark notifications as read using misskey.py
  • Modified parsing.py
    • Takes the client and notification as Input
    • Outputs a list with command,full_user, arguments and note_obj
  • Changed Gacha_response.py to response.py
    • Takes the list coming from parsin.py
    • Outputs response
Referencing issue #3 * Modified the bot_app.py process * Notifications don't get sent to the parser if they aren't a mention * note variables more descriptive * Response gets sent directly from bot_app.py * Visibility gets checked here too instead of going inside of the function * Sleep time lowered to two so testing is faster * There may be a way to mark notifications as read using misskey.py * Modified parsing.py * Takes the client and notification as Input * Outputs a list with command,full_user, arguments and note_obj * Changed Gacha_response.py to response.py * Takes the list coming from parsin.py * Outputs response
waifu added 2 commits 1 day ago
waifu requested review from nai 1 day ago
waifu added the
Refactoring
label 23 hours ago

Reviewers

nai was requested for review 1 day ago
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: waifu/kemoverse#11
Loading…
There is no content yet.