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

fix:Change Email Sync Status Display from Complete to Active #5980

Closed
wants to merge 1 commit into from
Closed

fix:Change Email Sync Status Display from Complete to Active #5980

wants to merge 1 commit into from

Conversation

Avnish-oP
Copy link

changed the label of syncStatusOption from complete to active

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

  • Updated sync status label from 'Complete' to 'Active' in useGetSyncStatusOptions.ts
  • Ensure all instances of 'Complete' are now displayed as 'Active'

1 file(s) reviewed, no comment(s)

Copy link

Welcome!

Hello there, congrats on your first PR! We're excited to have you contributing to this project.
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.

Generated by 🚫 dangerJS against 2ce3563

@charlesBochet
Copy link
Member

Hi @Avnish-oP, thanks for the PR, is there any related issue or is it a personal intiative? (in case we will need a review from design)

@charlesBochet
Copy link
Member

Found it: #5961
It seems to be already assigned but the contributor has not raised a PR yet.
Also we already have another PR:
#5965

@charlesBochet
Copy link
Member

@Avnish-oP as there is a previous PR and you were not assigned on the issue, I'll close this one, sorry about that.

Feel free to work on any other good first issue but make sure to be assigned to it first :)

@Avnish-oP
Copy link
Author

@Avnish-oP as there is a previous PR and you were not assigned on the issue, I'll close this one, sorry about that.

Feel free to work on any other good first issue but make sure to be assigned to it first :)

yes i will remeber it for the future ;)

@Avnish-oP Avnish-oP deleted the Avnish branch June 23, 2024 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants