Skip to content

statushistory_model#138

Merged
jirhiker merged 3 commits into
stagingfrom
statushistory_model
Sep 17, 2025
Merged

statushistory_model#138
jirhiker merged 3 commits into
stagingfrom
statushistory_model

Conversation

@ksmuczynski
Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem / context:

  • Need a central, polymorphic log for all time-variant operational statuses (e.g., Use Status, Access Status)

How

Implementation summary - the following was changed / added / removed:

  • New status_history.py file was created to establish relevant fields.
  • A new HasStatusHistory mixin was added to the db/base.py file.
  • Unrelated to the Jira ticket, docstrings were added to the db/base.py file to summarize the contents of the file and describe the mixin functionality.

Notes

Any special considerations, workarounds, or follow-up work to note?

  • The HasStatusHistory mixin has not yet been added to/inherited by any primary or core tables.

…b/base.py file and its mixins.

feat: add StatusHistory model to log all time-variant operational statuses.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 16, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
db/base.py 80.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
db/base.py 94.54% <80.00%> (ø)

Copy link
Copy Markdown
Contributor

@jacob-a-brown jacob-a-brown left a comment

Choose a reason for hiding this comment

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

some questions about the StatusHistory tab and mixin. I like the documentation you wrote a lot 👍

Comment thread db/status_history.py
Comment thread db/base.py Outdated
Comment thread db/base.py
@jirhiker jirhiker merged commit 343af2b into staging Sep 17, 2025
3 checks passed
@ksmuczynski ksmuczynski deleted the statushistory_model branch September 24, 2025 16:44
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