Resonate icon indicating copy to clipboard operation
Resonate copied to clipboard

Enhance]:Replace-deperacted-OnWillPopScope-with-PopScope

Open kareemabdeen opened this issue 1 year ago • 3 comments

Description

Now, instead of using the deprecated OnWillPopScope, you can simply use PopScope to achieve the same functionality in a more efficient and future-proof way, also Refactored save changes Dialog as separated function in the app utils files to be separated from the ui code

Fixes #270

Type of change

Please delete options that are not relevant.

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [x] Refactor (does not change functionality, e.g. code style improvements, linting)
  • [ ] Documentation update

How Has This Been Tested?

with my emulator

Please include screenshots below if applicable. here is how app functions after the changes had done https://github.com/AOSSIE-Org/Resonate/assets/118139061/ced336de-b05b-4af5-8386-18b1bd84f220

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes
  • [ ] Any dependent changes have been merged and published in downstream modules
  • [x] I have checked my code and corrected any misspellings

Maintainer Checklist

  • [ ] closes #xxxx (Replace xxxx with the GitHub issue number)
  • [x] Tag the PR with the appropriate labels

kareemabdeen avatar Mar 11 '24 22:03 kareemabdeen

@chandansgowda if there's any problem on the branch please let me know

kareemabdeen avatar Mar 27 '24 20:03 kareemabdeen

Hey @kareemabdeen, This is good to merge will merge once the PR prior to this are resolved

Aarush-Acharya avatar May 10 '24 15:05 Aarush-Acharya

Hey @kareemabdeen this is already taken care of in #274

Aarush-Acharya avatar May 13 '24 14:05 Aarush-Acharya

Hey, @Aarush-Acharya It is a duplicate, but the original PR has some conflicts and the contributor is not responding so I am merging this. I am removing the duplicate tag and also closing the original PR without merging.

Good work @kareemabdeen 🚀

vrundraval24 avatar May 23 '24 16:05 vrundraval24