Added verifyBeforeUpdateEmail method to Web, iOS, Android

[ewwwgiddings]

Pull request to handle this feature request:
#739

Pull request checklist

Please check if your PR fulfills the following requirements:

• The changes have been tested successfully.
• A changeset has been created (npm run changeset).
• I have read and followed the pull request guidelines.

[robingenz]

This already looks very good. Please take a look at my comments.

[ewwwgiddings]

Thanks! Wasn't sure how far I was off with those.

I just implemented all your suggestions now (with just the one thought about handling exceptions elsewhere as well) and hit the re-request review button.

[ewwwgiddings]

I also just updated the newEmail definition to what you liked and added an example in the README.

[ewwwgiddings]

Just pushed a commit that adds the callback like the example method.

[ewwwgiddings]

Just fixed the prettier issues as well. There were around 25 in unrelated files when using --write on the prettier command as a side note for some time in the future maybe.

[robingenz]

These should be the last changes.

[ewwwgiddings]

Changes pushed

[pkg-pr-new]

Open in Stackblitz

@capacitor-firebase/analytics

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/analytics@741

@capacitor-firebase/app

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/app@741

@capacitor-firebase/app-check

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/app-check@741

@capacitor-firebase/authentication

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/authentication@741

@capacitor-firebase/crashlytics

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/crashlytics@741

@capacitor-firebase/firestore

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/firestore@741

@capacitor-firebase/functions

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/functions@741

@capacitor-firebase/messaging

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/messaging@741

@capacitor-firebase/performance

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/performance@741

@capacitor-firebase/remote-config

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/remote-config@741

@capacitor-firebase/storage

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/storage@741

commit: 8de3677

[robingenz]

@ewwwgiddings Great, thank you! Two points remain before we can merge:

1. Create a changeset with the commandnpm run changeset. Please use the following text: feat: add `verifyBeforeUpdateEmail` method
2. Test your implementation using this dev release:

   npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/authentication@741
   

[ewwwgiddings]

Pushed the changeset. As far as testing I'm currently unable to test anything but web with my setup. Eventually I'll get to a point I have all the environments set up but atm it's probably a couple months away.

[robingenz]

@ewwwgiddings No problem. I will test it on Android and iOS before the next release. Thank you for your contribution!