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

feat(#1095): Update data sync documentation #1243

Merged
merged 14 commits into from
Dec 4, 2023
Merged

Conversation

Benmuiruri
Copy link
Contributor

Description

Document the current sync strategy on the docs site. The documentation has been updated with information that cover the following topics:

  • data replication
  • sync status states
  • sync intervals
  • sync retry mechanism

#1095

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

@Benmuiruri Benmuiruri marked this pull request as ready for review November 27, 2023 12:43
@Benmuiruri
Copy link
Contributor Author

Hi @garethbowen , my first draft of updating our Sync documentation is ready for your review to fact-check my understanding of syncing. The write up is based on the discussion between Michael and Jenni

Let me know if there's anything I could add, delete, or correct.

Thanks.

Copy link
Member

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

Wow! Thanks for writing all this up!

content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
Copy link
Member

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

Nice!

I have two more suggestions, and two left over from the original review.

content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
content/en/apps/concepts/navigation.md Outdated Show resolved Hide resolved
Copy link
Member

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

Great work! Ship it!

@Benmuiruri Benmuiruri merged commit 05affd3 into main Dec 4, 2023
2 checks passed
@Benmuiruri Benmuiruri deleted the chore-update-sync branch December 4, 2023 07:15
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.

2 participants