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.

Sign up for GitHub

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

Jump to bottom

Refactor namespacing to better fit our current design #496

Open
stuartc opened this issue Dec 12, 2022 · 0 comments
Open

Refactor namespacing to better fit our current design #496

stuartc opened this issue Dec 12, 2022 · 0 comments

Comments

@stuartc
Copy link
Member

stuartc commented Dec 12, 2022

Details

Since we introduced the WorkOrder module, the existence of Invocation doesn't fit anymore.
If anything, I propose the follow models are moved under WorkOrder.

  • Run
  • Attempt
  • AttemptRun
  • Dataclip
  • ...

Implementation notes

I also think we should revert the spelling of WorkOrder to Workorder, the underscore variable naming is tedious and used in so many places that it's visually unnecessary.

Release notes

User acceptance criteria
@taylordowns2000 taylordowns2000 moved this to Icebox in v2 Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
v2
Status: Icebox
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stuartc