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

BAU: enable app level logging #82

Merged
merged 2 commits into from
Feb 6, 2024
Merged

BAU: enable app level logging #82

merged 2 commits into from
Feb 6, 2024

Conversation

gidsg
Copy link
Contributor

@gidsg gidsg commented Feb 5, 2024

Change description

Previously app level logging was not enabled which meant that only server level gunicorn logs were outputted on the hosted environments.
This enables logging using the existing logging functionality in utils

  • Unit tests and other appropriate tests added or updated
  • README and other documentation has been updated / added (if needed)
  • Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test

App should show app level logs when running locally

Screenshots of UI changes (if applicable)

Copy link
Contributor

@cyrusdobbs cyrusdobbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice spot 🌮

@gidsg gidsg merged commit e6a14f9 into main Feb 6, 2024
2 checks passed
@gidsg gidsg deleted the enable-logging branch February 6, 2024 11:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants