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

Migrate getVersion GraphQL Request to REST #2221

Open
1 task
Huongg opened this issue Dec 5, 2024 · 5 comments
Open
1 task

Migrate getVersion GraphQL Request to REST #2221

Huongg opened this issue Dec 5, 2024 · 5 comments

Comments

@Huongg
Copy link
Contributor

Huongg commented Dec 5, 2024

Description

As part of our ongoing effort to remove all GraphQL dependencies related to Experiment Tracking, we found that the getVersion GraphQL request is in the flowchart. This task involves migrating the getVersion request to use REST so that we can remove GraphQL from the codebase entirely.

Context

Transitioning to REST helps to simplify the architecture and remove GraphQL from the codebase completely

Checklist

  • Include labels so that we can categorise your feature request
@rashidakanchwala
Copy link
Contributor

is this a part of the remove ET PR or is it something we plan to do before we merge the remove ET PR ?

@Huongg
Copy link
Contributor Author

Huongg commented Jan 9, 2025

hey @rashidakanchwala this is just a tech debt in general. It's related to getVersion in flowchart. It doesn't block ET removal

@rashidakanchwala
Copy link
Contributor

but if we remove graphQL entirely, we won't be able to getVersion right?

@Huongg
Copy link
Contributor Author

Huongg commented Jan 13, 2025

hey @rashidakanchwala yeah that's correct. Therefore GraphQl is not fully removed yet. So far we only removed the part that related to ET. In order to remove graphQl completely, we need to migrate the Graphql to Rest, which is this ticket

@rashidakanchwala
Copy link
Contributor

rashidakanchwala commented Jan 13, 2025

ok, great. I suggest let's do this small migration in the following sprint. Then when we remove ET, we remove entire GraphQL and all dependencies as well. Wdyt?

@rashidakanchwala rashidakanchwala moved this from Inbox to Backlog in Kedro-Viz Jan 13, 2025
@rashidakanchwala rashidakanchwala moved this from Backlog to Todo in Kedro-Viz Jan 13, 2025
@Huongg Huongg moved this from Todo to In Progress in Kedro-Viz Jan 15, 2025
@Huongg Huongg moved this from In Progress to In Review in Kedro-Viz Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Review
Development

No branches or pull requests

2 participants