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

docs: Add n8n backup automation example guide #3648

Merged
merged 16 commits into from
Jun 12, 2024
Merged

docs: Add n8n backup automation example guide #3648

merged 16 commits into from
Jun 12, 2024

Conversation

greirson
Copy link
Contributor

@greirson greirson commented May 25, 2024 •
edited
Loading

What type of PR is this?

  • documentation

What this PR does / why we need it:

Add guide for leveraging n8n to automate backups

@greirson greirson changed the title n8n-backup-example docs: Add n8n backup automation example guide May 25, 2024
@greirson
Copy link
Contributor Author

@hay-kot for the n8n-mealie-backup.json file to import into n8n, where should this file live?

@boc-the-git
Copy link
Collaborator

@hay-kot for the n8n-mealie-backup.json file to import into n8n, where should this file live?

I said on Discord:

We don't really have a precedent.. I think what you've done is probably fine. The only alternative that comes to mind is making the folder json, but it seems quite niche and unnecessary.

@greirson greirson marked this pull request as ready for review May 26, 2024 18:44
boc-the-git
boc-the-git requested changes Jun 2, 2024
View reviewed changes
Copy link
Collaborator

@boc-the-git boc-the-git left a comment

Choose a reason for hiding this comment

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

Some minor corrections, but otherwise think this is fine to merge.

docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Outdated Show resolved Hide resolved
docs/docs/documentation/community-guide/n8n-backup-automation.md Show resolved Hide resolved
greirson and others added 3 commits June 2, 2024 16:53
@greirson @boc-the-git
Apply suggestions from code review
d05eaac 
Co-authored-by: boc-the-git <3479092+boc-the-git@users.noreply.github.com>
@greirson
adding missing image
6bb6c49
@greirson
Merge branch 'mealie-next' into n8n-backup
eaf7d32
@greirson greirson requested a review from boc-the-git June 4, 2024 19:10
boc-the-git
boc-the-git approved these changes Jun 12, 2024
View reviewed changes
Copy link
Collaborator

@boc-the-git boc-the-git left a comment

Choose a reason for hiding this comment

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

LGTM!

Sorry @greirson, I should've got back to this sooner than what I have.

@boc-the-git boc-the-git enabled auto-merge (squash) June 12, 2024 09:36
@boc-the-git boc-the-git merged commit b3241d3 into mealie-recipes:mealie-next Jun 12, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boc-the-git boc-the-git boc-the-git 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
@greirson @boc-the-git