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

Jetpack: add SIWA to 18.9 #17693

Merged
merged 5 commits into from
Dec 21, 2021
Merged

Jetpack: add SIWA to 18.9 #17693

merged 5 commits into from
Dec 21, 2021

Conversation

leandroalonso
Copy link
Contributor

SIWA was added to 18.8 on #17671 and #17675. This PR simply cherry-picks the commits to 18.9.

To test

  1. Make sure CI is green
  2. Once Testflight version is released, test the SIWA flow

@leandroalonso leandroalonso added this to the 18.9 ❄️ milestone Dec 20, 2021
@leandroalonso leandroalonso self-assigned this Dec 20, 2021
@peril-wordpress-mobile
Copy link

You can trigger an installable build for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

Copy link
Contributor

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

Diffs match and CI is green 👍

I'm going to merge this immediately to ship a new beta and test the flow.

@mokagio mokagio merged commit c4f010e into release/18.9 Dec 21, 2021
@mokagio mokagio deleted the task/add_siwa_to_18_9 branch December 21, 2021 04:40
@mokagio
Copy link
Contributor

mokagio commented Dec 21, 2021

@leandroalonso this has been bundled as part of 18.9 beta 2 (18.9.0.2).

Thanks for your work 🙌

Once Testflight version is released, test the SIWA flow

The 18.9.0.2 build is now available. I tested signing in with email forwarding and it "worked", meaning it showed me the "no sites" screen, as expected. I don't have a SIWA account setup, so I haven't been able to test that particular flow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants