ConfrerieDuKassoulait/Botanique
ConfrerieDuKassoulait/Botanique
2
0
Fork 0
Code Issues 26 Pull requests 2 Projects 1 Releases Packages 1 Activity Actions

feat: Reminders #44

Merged
Anri merged 54 commits from feat/reminders into main 2023-01-17 12:15:15 +01:00
Conversation 4 Commits 54 Files changed 27 +3659 -588
Anri commented 2022-07-27 21:48:43 +02:00
Owner
Copy link

Checklist:

  • Suivre les indications de CONTRIBUTING.md
  • Référence aux tickets (par exemple Closes #xyz)

Closes #1

Additional changes:

Checklist: - [x] Suivre les indications de `CONTRIBUTING.md` - [x] Référence aux tickets (par exemple `Closes #xyz`) Closes #1 Additional changes: - use of ChatInputCommandInteraction (part of v14), af5fc917e57e6fc13a30001415d8a53ee8191c20 - fixed locales progress bars, c6120bf8b4680b512ae5fcf431dc7f44863129e6 - updates dependencies 1c20997703f5426206dff62036ee8bbf48682a8b b83a1d30537142569499eb0079128d88e6d1ddcc c0ef16b74c5762badd2665262e6af9a88680516c 4e38df8cc8e776ec689c19a1d84f62ed8b49d194 41a32f9d97baac0339a0f08cd75878aa03eabe7c - better handle of errors - support of modals - support of buttons https://git.kennel.ml/ConfrerieDuKassoulait/Botanique/compare/e6d8bd72659cd509ef54eaf019bc38595b446556...ffc395b4aa5160fc72c7430f43219334f1aab75d (excluding 4bac943011a39e14ed2aa422427824b80d2b314f)
* Cleanup docker image and update node from 16 to 18
* fix bar lenght by adding floor and ceil
* Add Modal support
* Use modal for reminder
* Add TODOs
* add locales
* add timeout when creating a reminder
Anri commented 2023-01-15 20:31:13 +01:00
Author
Owner
Copy link

Since this PR brings a lot of changes, the main part being already done, I propose to merge it and finish the missing part, namely the reminders list, later

Since this PR brings a lot of changes, the main part being already done, I propose to merge it and finish the missing part, namely the reminders list, later
Anri changed title from WIP: feat: Reminders to feat: Reminders 2023-01-15 20:31:15 +01:00
Anri commented 2023-01-16 08:37:04 +01:00
Author
Owner
Copy link

Since this PR brings a lot of changes, the main part being already done, I propose to merge it and finish the missing part, namely the reminders list, later

Nervermind let's fully complete the PR

> Since this PR brings a lot of changes, the main part being already done, I propose to merge it and finish the missing part, namely the reminders list, later Nervermind let's fully complete the PR
Anri changed title from feat: Reminders to WIP: feat: Reminders 2023-01-16 08:37:13 +01:00 
This reverts commit b70f4e5680.
* add a page arg
Anri commented 2023-01-16 12:06:47 +01:00
Author
Owner
Copy link

Note that the button implementation differ from other interactions since the update need to be done via the collector

Note that the button implementation differ from other interactions since the update need to be done via the [collector](https://discordjs.guide/interactions/buttons.html#updating-the-button-message)
* add message when the list is empty
* wip: add button to update when the list is empty
* fix reminder deletion
Anri changed title from WIP: feat: Reminders to feat: Reminders 2023-01-16 19:14:08 +01:00
Anri commented 2023-01-16 19:14:31 +01:00
Author
Owner
Copy link

Ready for review

Ready for review
* add: buttons and modals
Anri merged commit 8f096b9589 into main 2023-01-17 12:15:15 +01:00
Anri referenced this pull request from a commit 2023-01-17 12:15:15 +01:00
feat: Reminders (#44)
Anri deleted branch feat/reminders 2023-01-17 12:15:15 +01:00
Anri referenced this pull request from a commit 2023-01-17 14:18:54 +01:00
feat: Reminders (#44)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
good first issue
Issues for getting started

  
help wanted
Need some help

  
invalid
Something is wrong

  
locale
en
 Issues about EN localization

  
locale
fr
 Issues about FR localization

  
migration
Migration from KassouBot

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
good first issue
help wanted
invalid
locale
en
locale
fr
migration
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ConfrerieDuKassoulait/Botanique!44
No description provided.