-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
feat: Add graphql to unified-desktop branch #17305
feat: Add graphql to unified-desktop branch #17305
Conversation
Thanks for taking the time to open a PR!
|
Test summaryRun details
View run in Cypress Dashboard ➡️ FlakinessThis comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
…ithub.com/cypress-io/cypress into tgriesser/unified-desktop-gui/add-graphql
@tgriesser I will pick this up and get it green so we can merge it into |
@lmiller1990 I had been starting to work on getting this so we can mount "fragments" for testing: 21206bd and adding SDL definition directives to Nexus so that we can specify what the "test values" should be in the server definition in a way that the frontend schema can consume. Unfortunately the timing on this work hasn't mapped well to my PTO this week and last. So feel free to pick it up from here for now - also was having trouble seeing how Vue validates props, either with the |
@tgriesser nice to see the test scaffolding. I'll get this back to ✅ , always good to keep things passing before it goes too out of sync. As for Vue, I've found you can use The prop |
Begins the process of adding graphql to the
unified-desktop-gui
branch:Adds:
yarn dev:watch
from the root for a server that restarts when GraphQL code is updatedyarn codegen --watch
from the launchpad appTodo:
yarn watch
PR Tasks
cypress-documentation
?type definitions
?cypress.schema.json
?