Skip to content

Added persist to the DataSession model - #137

Open
jnation3406 wants to merge 1 commit into
mainfrom
feature/persist_sessions
Open

jnation3406 wants to merge 1 commit into
mainfrom
feature/persist_sessions

Conversation

@jnation3406

Copy link
Copy Markdown
Contributor
  • Adds a persist boolean field to the DataSession model defaulting to False.
  • Adds a management command to delete all data sessions with persist=False if they were created > 30 days ago.
  • Adds a cronjob to the k8s stuff to call the management command once a day
  • Adds a new response_fields param to requests for DataSessions which, if specified, limits the returned fields to only those specified. The default without it specified is to return all fields. This is because we have a tremendous amount of data in the operations and input_data fields of DataSessions which we don't want or need returned when just sending a PATCH request to update persist.

…ment command to delete all non-persistant datasessions after 30 days

@sfoale sfoale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

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.

2 participants