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

Refactor updating status of Sync Destinations and Opt Attributes #1035

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Refactor updating status of Sync Destinations and Opt Attributes #1035

merged 1 commit into from
Oct 31, 2024

Conversation

RuiChen12
Copy link
Contributor

@RuiChen12 RuiChen12 commented Sep 27, 2024 •
edited
Loading

Ticket Link

1800

List of squashed commits

  • updated the return type for updateSyncDest

Test Checklist

  • Exhibits Clear Code Structure:
  • Project Unit Tests Passed:
  • Project Jasmine Tests Passed:
  • Executes Expected Functionality:
  • Tested For Incorrect/Unexpected Inputs:

@RuiChen12 RuiChen12 requested a review from JorWo September 27, 2024 11:15
@RuiChen12 RuiChen12 changed the title Combining enableSyncDest and disableSyncDest into updateSyncDest in groupings-api Refactor updating status of Sync Destinations and Opt Attributes Oct 30, 2024
@JorWo JorWo merged commit 71bac76 into main Oct 31, 2024
14 checks passed
RuiChen12 added a commit that referenced this pull request Nov 27, 2024
@RuiChen12
Refactor updating status of Sync Destinations and Opt Attributes (#1035)
c9ab528
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fduckart fduckart fduckart left review comments

@JorWo JorWo JorWo 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.
3 participants
@RuiChen12 @fduckart @JorWo