v2 changes of pipeline and api#467
Conversation
hari-kuriakose
left a comment
There was a problem hiding this comment.
@muhammad-ali-e Models look good.
We will take care of duplication when we delete v1 eventually, and also migration files can be generated later together.
athul-rs
left a comment
There was a problem hiding this comment.
LGTM.
@muhammad-ali-e Hope you have tried triggering a pipeline execution since those also involve organisation checks during runs.
@athul-rs We can check it once it merged. I think it handled by the |
|
|
Co-authored-by: Hari John Kuriakose <hari@zipstack.com>




NOTE
What
Why
How
get_organizationmethod to utilize the user context.apps.py.modelswith updated table names.DefaultOrganizationMixinclass.APIDeploymentModelManagerwithDefaultOrganizationManagerMixinto get the organization data inapi.PipelineModelManagerwithDefaultOrganizationManagerMixinto get the organization data inpipeline.Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
Database Migrations
Env Config
Relevant Docs
Related Issues or PRs
Dependencies Versions
Notes on Testing
Screenshots
Checklist
I have read and understood the Contribution Guidelines.