Changes from development #125
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
got the functionality for filtering by user, group and state working
now displayes result of database query can view simply or detailed can now order the in descending or ascending by creation time cleaned up the code and changed some variable names to be better
there are now options with start and end dates as well as a select by id choice you can now also filter by record state
Sundered the query_monitor_db function into one that opens the database and another that queries it (which makes it easier to mock the database for testing) added a filter by transaction_id that has the same functionality as filtering by id added displaying group to simple view
currently mocks the monitor database and has a way to put records in it to be queried by the test code
It still needs to be checked for formatting using black
added a message to each pytest to explain briefly what it is ran black to format the files
moved user error checking to another function updated that formatting function to use SystemExit and and added "Error:" to the front of two of the messages updated the pytests to account for the error message and code success changes moved create message to another function added a function to turn the monitor models into dictionaries to make it easier in the FastAPI web page updated the complex and simple prints for the dictionary
Created command line tool to output status of all jobs in NLDS
…ories that belong to NLDS user
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.