Skip to content
This repository has been archived by the owner on Jul 22, 2021. It is now read-only.

Warnings log across different servers. #7

Closed
ghost opened this issue Apr 13, 2021 · 4 comments · Fixed by #9
Closed

Warnings log across different servers. #7

ghost opened this issue Apr 13, 2021 · 4 comments · Fixed by #9
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@ghost
Copy link

ghost commented Apr 13, 2021

If you gain a warning in one server, the warning carries between servers rather than remaining in the server you were warned in.
We need to ensure that warnings do not carry across servers.

@ghost ghost added help wanted Extra attention is needed good first issue Good for newcomers labels Apr 13, 2021
@ghost ghost added this to the Minimal Bot release milestone Apr 13, 2021
@ghost ghost mentioned this issue Apr 13, 2021
@ghost ghost added the bug Something isn't working label Apr 13, 2021
@OfekShochat
Copy link
Collaborator

Possible Implementation and explanation:
We use a local file called warnings.json to log warnings. So if you execute the same bot in the same directory it will read that warnings.json.
This can be Implemented with guild Id also as an additional key and then a sub json will be the warnings to that server/guild

@ghost
Copy link
Author

ghost commented Apr 13, 2021

Possible Implementation and explanation:
We use a local file called warnings.json to log warnings. So if you execute the same bot in the same directory it will read that warnings.json.
This can be Implemented with guild Id also as an additional key and then a sub json will be the warnings to that server/guild

Do you think you would be able to work on this and fix it?

@OfekShochat
Copy link
Collaborator

Working on it :)

@ghost
Copy link
Author

ghost commented Apr 18, 2021

Thank you!

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant