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

TP2000-1590 TaskWorkflow summary info support #1337

Conversation

paulpepper-trade
Copy link
Collaborator

TP2000-1590 TaskWorkflow summary info support

Why

TaskWorkflow summary information is required to present a workflow's information in a Task-like way.

TaskWorkflow summary info must be presentable alongside Task info, in order to support at-a-glance list views that combine the two types of information.

What

This PR:

  • Adds a summary Task instance to the Workflow model, referenced by its Workflow.summary_task field.
  • Includes migrations to create the new field in the database and populate with dummy data. The dummy data is only a consideration while this work is in development, since workflows will be newly introduced and migrations squashed before moving to the master, main line.
  • Adds three new filter functions to TaskQueryset: non_workflow(), workflow_summary() and top_level(). These should cater for the need to extract the Task instances that provide summary information for various views in a follow-up PR.
  • Adds and updates model factories and fixtures for use in related tests.
  • Provides unit tests for new queryset filtering.

Checklist

  • Requires migrations? No
  • Requires dependency updates? No

@paulpepper-trade paulpepper-trade requested a review from a team as a code owner November 21, 2024 17:55
@paulpepper-trade paulpepper-trade merged commit 7dfe059 into TP2000-1471--task-workflow Nov 25, 2024
@paulpepper-trade paulpepper-trade deleted the TP2000-1580--task-workflow-combined-list-view branch November 25, 2024 17:21
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