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

[pull] development from modmail-dev:development #39

Merged
merged 11 commits into from
Nov 20, 2023

Conversation

pull[bot]
Copy link

@pull pull bot commented Nov 20, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

nullishamy and others added 11 commits November 19, 2023 05:45
* Add JSON logging support

This adds support for JSON logging, along with the relevant options required.

This does not change the default log behaviour, so should be backwards compatible.

It is opt in via the LOG_FORMAT option, which can be 'json' to use the new logger, or anything else to fallback
to the old behaviour.

This is implemented in terms of a custom formatter, which is optionally applied to the stdout stream.
The debug stream is unaffected by this.

* Allow JSON to be selected when creating handlers

* Allow different formats to be selected for streams/files

* Remove old / unused code

* Add new config opts to helpfile

* Formatting, basic typing and reorder for consistency in project.

---------

Co-authored-by: Jerrie-Aries <[email protected]>
Co-authored-by: Taku <[email protected]>
* Fix rate limit issue on raw reaction add/remove events.

* Pasd message object to `find_linked_messages` since it is already fetched.

---------

Co-authored-by: Taku <[email protected]>
* Add config to change how long logs are stored

* Optimize deleting logs

* Update core/config_help.json

Co-authored-by: khakers <[email protected]>
Signed-off-by: Cordila <[email protected]>

* Testing changes

* Update core/config_help.json (Fix copy paste)

Co-authored-by: Jerrie <[email protected]>
Signed-off-by: Cordila <[email protected]>

---------

Signed-off-by: Cordila <[email protected]>
Co-authored-by: Cordila <[email protected]>
Co-authored-by: khakers <[email protected]>
Co-authored-by: Taku <[email protected]>
Co-authored-by: Jerrie <[email protected]>
* Add `rename` to `registry.json`

Signed-off-by: Nicklaus <[email protected]>

* Change icon

Signed-off-by: Nicklaus <[email protected]>

---------

Signed-off-by: Nicklaus <[email protected]>
* fix: discord invite ( new server )

Signed-off-by: lorenzo132 <[email protected]>

* Update link

Signed-off-by: Taku <[email protected]>

---------

Signed-off-by: lorenzo132 <[email protected]>
Signed-off-by: Taku <[email protected]>
Co-authored-by: Taku <[email protected]>
@pull pull bot added the ⤵️ pull label Nov 20, 2023
@pull pull bot merged commit 53d40e0 into WFP-Doobelepers:development Nov 20, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants