Skip to content

website/blog: sync objects in external databases #2099

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
Nov 2, 2021
Merged

website/blog: sync objects in external databases #2099

merged 1 commit into from
Nov 2, 2021

Conversation

@a8m
Copy link
Member

@a8m a8m commented Nov 1, 2021 •
edited
Loading

yonidavidson
yonidavidson approved these changes Nov 1, 2021
View reviewed changes
Copy link
Collaborator

@yonidavidson yonidavidson left a comment

Choose a reason for hiding this comment

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

I like it!

masseelch
masseelch approved these changes Nov 1, 2021
View reviewed changes
Copy link
Collaborator

@masseelch masseelch left a comment

Choose a reason for hiding this comment

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

Much like. Very good!

doc/website/blog/2021-11-1-sync-objects-in-external-database.md

### Schema Hooks

[Hooks](https://entgo.io/docs/hooks) are a powerful feature of Ent that allows adding custom logic before and after
Copy link
Collaborator

@masseelch masseelch Nov 1, 2021

Choose a reason for hiding this comment

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

Suggested change
[Hooks](https://entgo.io/docs/hooks) are a powerful feature of Ent that allows adding custom logic before and after
[Hooks](https://entgo.io/docs/hooks) are a powerful feature of Ent that allow adding custom logic before and after

hilakashai
hilakashai reviewed Nov 1, 2021
View reviewed changes
rotemtam
rotemtam requested changes Nov 1, 2021
View reviewed changes
Copy link
Collaborator

@rotemtam rotemtam left a comment

Choose a reason for hiding this comment

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

Not sure about the post title. Let's discuss

@a8m a8m force-pushed the blog/sync-external-objects branch 2 times, most recently from 620eb89 to 191180d Compare November 1, 2021 21:53
hedwigz
hedwigz approved these changes Nov 2, 2021
View reviewed changes
Copy link
Contributor

@hedwigz hedwigz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Great piece 🚀

@a8m a8m force-pushed the blog/sync-external-objects branch from 191180d to a910869 Compare November 2, 2021 12:03
@a8m
Copy link
Member Author

a8m commented Nov 2, 2021

Thanks all for the review 🙏 ❤️

@a8m a8m merged commit d0118b2 into master Nov 2, 2021
@a8m a8m deleted the blog/sync-external-objects branch November 2, 2021 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rotemtam rotemtam rotemtam requested changes

@masseelch masseelch masseelch approved these changes

+3 more reviewers

@hilakashai hilakashai hilakashai left review comments

@yonidavidson yonidavidson yonidavidson approved these changes

@hedwigz hedwigz hedwigz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@a8m @rotemtam @yonidavidson @hedwigz @masseelch @hilakashai