Skip to content
This repository has been archived by the owner on Mar 28, 2024. It is now read-only.

Improve logging #83

Open
elenirotsides opened this issue Jun 22, 2021 · 8 comments
Open

Improve logging #83

elenirotsides opened this issue Jun 22, 2021 · 8 comments
Labels
3 enhancement New feature or request Feature good first issue Good for newcomers

Comments

@elenirotsides
Copy link
Owner

Is your feature request related to a problem? Please describe.
We need a means of logging...we currently have nothing. I'd like to be able to track where and when problems occur, should they occur.

@elenirotsides elenirotsides added enhancement New feature or request good first issue Good for newcomers 3 Feature labels Jun 22, 2021
@centanomics
Copy link

Hi can i be assigned to this issue? I've built a few discord bots before and might be able to help you out.

Also where are you looking to log this information? In the console or somewhere else?

@elenirotsides
Copy link
Owner Author

Hi!! @centanomics I apologize for the delay in my response! I haven't been getting notifications for some reason and I need to look into my issue so I don't miss anymore messages. Yes of course! I just refactored the codebase though so definitely take a look and see if you're still interested in helping out. I'll assign you for now, let me know if you need anything!

I'm thinking maybe we can log info to the console, but if you have any ideas or other suggestions, feel free to let me know!

@centanomics
Copy link

I think starting out with just logging information to the console is good. I think an idea for later down the line could be email notifications? No idea what they would be used for though.

@elenirotsides
Copy link
Owner Author

elenirotsides commented Aug 15, 2021

Yeah I totally agree. The only use case for email notifications that I can think of is if something catastrophically fails, but I don't think we need to do this. Logging in the console sounds like a good start!

@centanomics
Copy link

Awesome, I'll get started on that then!

@elenirotsides
Copy link
Owner Author

Hi @centanomics are you still working on this? Please feel free to reach out if you have any questions.

@centanomics
Copy link

Yes I am, apologies as I've been really busy lately. I do have the class method created I just need to get around to actually implementing it

@elenirotsides
Copy link
Owner Author

That's great! And no worries. Looking forward to your PR 😁

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3 enhancement New feature or request Feature good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants