8000 add example edgeschema with privacy/hooks by amrnt · Pull Request #2743 · ent/ent · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add example edgeschema with privacy/hooks #2743

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
Merged
merged 1 commit into from
Jul 13, 2022
Merged

add example edgeschema with privacy/hooks #2743

merged 1 commit into from
Jul 13, 2022

Conversation

amrnt
Copy link
Contributor
@amrnt amrnt commented Jul 12, 2022 •
edited
Loading

Fixed by #2745

Add an example that reproduced the issue before

Fixing error:

2022/07/12 23:27:46 running ent codegen: entc/load: # entgo.io/ent/entc/integration/edgeschema/ent
./user_create.go:328:7: createE.defaults() (no value) used as value
./user_create.go:351:7: createE.defaults() (no value) used as value
./user_create.go:374:7: createE.defaults() (no value) used as value
./user_create.go:420:7: createE.defaults() (no value) used as value
./user_create.go:443:7: createE.defaults() (no value) used as value
./user_update.go:477:7: createE.defaults() (no value) used as value
./user_update.go:500:7: createE.defaults() (no value) used as value
./user_update.go:523:7: createE.defaults() (no value) used as value
./user_update.go:543:7: createE.defaults() (no value) used as value
./user_update.go:566:7: createE.defaults() (no value) used as value
./user_update.go:566:7: too many errors
exit status 1
ent/generate.go:7: running "go": exit status 1

@amrnt amrnt force-pushed the m2m-edgeschema-bug branch from fa2e986 to f0afbe3 Compare July 12, 2022 21:56
@a8m
Copy link
Member
a8m commented Jul 13, 2022

Hey @amrnt, please see the fix here.

If you still want to add these example schemas to the integration tests, please rebase this PR on #2745. Thanks 🙏

@amrnt amrnt force-pushed the m2m-edgeschema-bug branch from f0afbe3 to 34398a6 Compare July 13, 2022 16:08
@amrnt amrnt changed the title fix edgeschema with privacy/hooks add example edgeschema with privacy/hooks Jul 13, 2022
@amrnt
Copy link
Contributor Author
amrnt commented Jul 13, 2022

@a8m Added an example

a8m
a8m approved these changes Jul 13, 2022
View reviewed changes
Copy link
Member
@a8m a8m left a comment

Choose a reason for hiding this comment

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

🚀

@a8m a8m merged commit 4491610 into ent:master Jul 13, 2022
@amrnt amrnt deleted the m2m-edgeschema-bug branch July 13, 2022 19:21
@timoha timoha mentioned this pull request Oct 7, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a8m a8m a8m approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amrnt @a8m
0