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

Migrate pipelines queries to Drizzle #140

Open
dinmukhamedm opened this issue Nov 2, 2024 · 1 comment
Open

Migrate pipelines queries to Drizzle #140

dinmukhamedm opened this issue Nov 2, 2024 · 1 comment
Assignees
@alviskalev
Labels
backend Issue with the app-server package enhancement New feature or request frontend Requires work in the frontend package good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dinmukhamedm
Copy link
Member

dinmukhamedm commented Nov 2, 2024 •
edited
Loading

Migrate get pipelines, create pipeline, rename pipeline, delete pipeline, update pipeline version, commit pipeline version, set target pipeline, etc. from app-server (routes/pipelines.rs and db/pipelines/*) directly to calling drizzle from the next routes in /api/projects/[projectId]/pipelines.

Be careful not to remove some db queries that are used by other parts of backend, e.g. get pipeline version graph for pipeline run requests.
@dinmukhamedm dinmukhamedm converted this from a draft issue Nov 2, 2024
@dinmukhamedm dinmukhamedm added enhancement New feature or request good first issue Good for newcomers frontend Requires work in the frontend package backend Issue with the app-server package help wanted Extra attention is needed labels Nov 2, 2024
@alviskalev
Copy link
Contributor

hi @dinmukhamedm could you please assign this issue to me?

@alviskalev alviskalev mentioned this issue Nov 23, 2024
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alviskalev alviskalev

Labels
backend Issue with the app-server package enhancement New feature or request frontend Requires work in the frontend package good first issue Good for newcomers help wanted Extra attention is needed
Projects
Drizzle migration
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dinmukhamedm @alviskalev