Skip to content

Conversation

ajay-odoogap
Copy link

Depends on #474

@lmignon
Copy link
Contributor

lmignon commented Jan 30, 2025

/ocabot migration graphql_demo

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jan 30, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 30, 2025
18 tasks
Copy link
Contributor

@lmignon lmignon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ajay-odoogap
Can you add in a separate and temporary commit a dependency on the #473 PR into the test-requirements.txt file as

odoo-addon-graphql-base @ git+https://github.com/OCA/rest-framework.git@refs/pull/473/head#subdirectory=graphql_base

This will allow the tests to be run.

@ajay-odoogap ajay-odoogap force-pushed the 18.0-mig-graphql_demo branch 2 times, most recently from 27d3da0 to 5806a80 Compare February 3, 2025 15:03
@ajay-odoogap ajay-odoogap force-pushed the 18.0-mig-graphql_demo branch from c0287ab to e5517bd Compare February 3, 2025 15:09
Copy link

github-actions bot commented Jun 8, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 8, 2025
@marielejeune
Copy link
Contributor

@lmignon Can you mark this PR as no stale please?

@lmignon lmignon added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Jun 23, 2025
@lmignon
Copy link
Contributor

lmignon commented Jun 23, 2025

@lmignon Can you mark this PR as no stale please?

It's done but some work is required to be able to merge this PR. Do you plan to finalise this PR?

@marielejeune
Copy link
Contributor

I don't think so. @ajay-odoogap do you plan to finish this PR ?


{
"name": "GraphQL Demo",
"version": "17.0.1.0.1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ajay-odoogap can you finish this PR? It's important so we can test it before merging graphql_base.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants