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

[go][graphql] fix node(id:ID) interface #29

Merged
merged 1 commit into from
Oct 25, 2023
Merged

[go][graphql] fix node(id:ID) interface #29

merged 1 commit into from
Oct 25, 2023

Conversation

yssk22
Copy link
Owner

@yssk22 yssk22 commented Oct 25, 2023

Summary

now we accept "helloproject" as an ID of Node, the code should check the value before passing it to ent.Noder()

Test

  • go test

Issue

  • N/A

@yssk22
[go][graphql] fix node(id:ID) interface
c3db875 
**Summary**

now we accept "helloproject" as an ID of Node, the code should check the value before
passing it to ent.Noder()

**Test**

- go test

**Issue**

- N/A
@yssk22 yssk22 temporarily deployed to ci-go October 25, 2023 04:32 — with GitHub Actions Inactive
@yssk22 yssk22 merged commit f53c5d3 into main Oct 25, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yssk22