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: Sync skip delete #696

Merged
merged 8 commits into from
Dec 3, 2024
Merged

feat: Sync skip delete #696

merged 8 commits into from
Dec 3, 2024

Conversation

murali-shris
Copy link
Member

@murali-shris murali-shris commented Oct 17, 2024

- What I did

  • introduced a flag skipDeletes in sync.
    - How I did it
  • modified syntax of sync:from verb
  • modified impl of sync verb handler to include skipDeletes flag.
    - How to verify it
  • tests should pass

@murali-shris murali-shris marked this pull request as ready for review October 22, 2024 06:53
@@ -1,6 +1,6 @@
name: at_commons
description: A library of Dart and Flutter utility classes that are used across other components of the atPlatform.
version: 5.0.1
version: 5.0.2
Copy link
Contributor

Choose a reason for hiding this comment

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

should be 5.1.0 as this is new feature

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@murali-shris murali-shris requested a review from gkc December 2, 2024 11:50
@@ -1,3 +1,5 @@
## 5.1.0
- feat: Introduce skipDeletes for sync:from verb
Copy link
Contributor

Choose a reason for hiding this comment

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

skipDeletesUntil

Copy link
Contributor

Choose a reason for hiding this comment

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

@murali-shris skipDeleteUntil

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@@ -9,6 +9,8 @@ class SyncVerbBuilder implements VerbBuilder {

bool isPaginated = false;

int? skipDeletesUntil;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please take this opportunity to add /// documentation for all of the parameters (commitId, regex, limit, isPaginated, skipDeletesUntil)

Copy link
Member Author

Choose a reason for hiding this comment

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

added the dart docs

@murali-shris murali-shris requested a review from gkc December 2, 2024 15:36
@@ -1,3 +1,5 @@
## 5.1.0
- feat: Introduce skipDeletes for sync:from verb
Copy link
Contributor

Choose a reason for hiding this comment

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

@murali-shris skipDeleteUntil

@murali-shris murali-shris merged commit 3af96be into trunk Dec 3, 2024
12 checks passed
@murali-shris murali-shris deleted the sync_skip_delete branch December 3, 2024 05:53
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