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.

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

Meilisync MySQL guide #2549

Merged
merged 6 commits into from
Sep 12, 2023
Merged

Meilisync MySQL guide #2549

merged 6 commits into from
Sep 12, 2023

Conversation

guimachiavelli
Copy link
Member

@guimachiavelli guimachiavelli commented Sep 7, 2023

Partially addresses #2542

Disclaimer for reviewers: I was not able to successfully install meilisync on my machine. I suspect it's related to some python misconfiguration on my end, but it became a huge time drain and I'm working on a short deadline for this. The guide might be rougher than usual because of that.

@guimachiavelli guimachiavelli added this to the September 2023 milestone Sep 7, 2023
@github-actions
Copy link

github-actions bot commented Sep 7, 2023

All broken links are now fixed, thank you!

@guimachiavelli guimachiavelli marked this pull request as ready for review September 11, 2023 11:47
@alallema
Copy link
Contributor

alallema commented Sep 11, 2023

@guimachiavelli,
It's not good news at all if it's complicated to install. In that case, would you like me to provide you with a code example for the return of certain commands?

Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

Congrats @guimachiavelli 🎉
I just put a comment but the rest is great for me

learn/primary_db_sync/meilisync_mysql.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@Strift Strift left a comment

Choose a reason for hiding this comment

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

I've left a few minor comments. overall it looks good to me

learn/primary_db_sync/meilisync_mysql.mdx Outdated Show resolved Hide resolved
learn/primary_db_sync/meilisync_mysql.mdx Outdated Show resolved Hide resolved
learn/primary_db_sync/meilisync_mysql.mdx Outdated Show resolved Hide resolved
@guimachiavelli
Copy link
Member Author

Thanks for the feedback, @alallema and @Strift.

I have addressed all of it, I believe, including the most important bit regarding docker-compose up not only building a docker image, but also running it.

Let me know if you spot anything else, otherwise please mark it as approved and I'll deploy it by the end of the day today.

Copy link
Contributor

@Strift Strift left a comment

Choose a reason for hiding this comment

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

Looks perfect to me!

@guimachiavelli guimachiavelli merged commit 1ff7590 into main Sep 12, 2023
2 checks passed
@guimachiavelli guimachiavelli deleted the meilisync-mysql branch September 12, 2023 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants