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

E2471: Reimplement logger #128

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

E2471: Reimplement logger #128

wants to merge 46 commits into from

Conversation

camhimes
Copy link

The goal of this project is to implement a logging mechanism for the new backend of the Expertiza system. This feature, similar to the one used in the older version of Expertiza, will log key application events to enhance traceability and debugging. The implementation will involve replicating the logging functionality found in the expertiza_logger.rb file, ensuring that the new backend is equipped with an efficient and maintainable logging system. A good starting point will be to log the various CRUD operations being performed to the database.

lgjenqui and others added 17 commits October 16, 2024 14:54
All other model classes inherit from AppRecord and will be logged.

Removed the logging statement from User as it inherits from AppRecord

Student_Task.rb is the only class that does not extend off of any class.
Added the logging to AppRecord.
…tudent unity id instead of invited student's
One post call to check for logging
Has not run locally
Added comments to describe Logger functionality
@camhimes camhimes marked this pull request as ready for review November 1, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants