Skip to content

feat: social login check added in description in create password. cp-7.57.0#21187

Merged
himanshuchawla009 merged 1 commit into
mainfrom
create_pasword_ui_changes
Oct 15, 2025
Merged

feat: social login check added in description in create password. cp-7.57.0#21187
himanshuchawla009 merged 1 commit into
mainfrom
create_pasword_ui_changes

Conversation

@grvgoel81

@grvgoel81 grvgoel81 commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

Description

  • social login check added in description in create password

fixes: #21192

Changelog

CHANGELOG entry: social login check added in description in create password

Related issues

Fixes: social login check added in description in create password

Manual testing steps

Feature:
1) Open the app in ios.
2)Create wallet using social login.
3)Observe changes in "description" below text "MetaMask password".

Screenshots/Recordings

Before

https://www.figma.com/design/pViOUcmjwhEzFsdrwknpNc/Onboarding?node-id=12501-102542&m=dev

After

social

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Show iOS-specific OAuth description only when OAuth login succeeded; add tests validating platform-specific text rendering.

  • ChoosePassword UI (app/components/Views/ChoosePassword/index.js):
    • Gate iOS-specific description behind Platform.OS === 'ios' && getOauth2LoginSuccess() so the iOS copy shows only after successful OAuth login; otherwise use the generic description. Android warning copy unchanged.
  • Tests (app/components/Views/ChoosePassword/index.test.tsx):
    • Import Platform and add tests to assert correct description text for iOS vs Android when oauthLoginSuccess is true.

Written by Cursor Bugbot for commit 2cbd2e5. This will update automatically on new commits. Configure here.

@grvgoel81 grvgoel81 requested a review from a team October 15, 2025 10:25
@grvgoel81 grvgoel81 self-assigned this Oct 15, 2025
@github-actions

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@grvgoel81 grvgoel81 changed the title feat: social login check added in title in create password. feat: social login check added in description in create password. Oct 15, 2025
@sonarqubecloud

Copy link
Copy Markdown

@himanshuchawla009 himanshuchawla009 changed the title feat: social login check added in description in create password. feat: social login check added in description in create password. cp-7.57.0 Oct 15, 2025
@himanshuchawla009 himanshuchawla009 added this pull request to the merge queue Oct 15, 2025
Merged via the queue into main with commit 4bab130 Oct 15, 2025
241 of 247 checks passed
@himanshuchawla009 himanshuchawla009 deleted the create_pasword_ui_changes branch October 15, 2025 12:07
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 15, 2025
@metamaskbot metamaskbot added release-7.58.0 Issue or pull request that will be included in release 7.58.0 release-7.57.0 Issue or pull request that will be included in release 7.57.0 and removed release-7.58.0 Issue or pull request that will be included in release 7.58.0 labels Oct 15, 2025
@metamaskbot

Copy link
Copy Markdown
Collaborator

Missing release label release-7.57.0 on PR. Adding release label release-7.57.0 on PR and removing other release labels(release-7.58.0), as PR was cherry-picked in branch 7.57.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-onboarding release-7.57.0 Issue or pull request that will be included in release 7.57.0 size-S team-onboarding Onboarding team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

iOS “Create Password” screen shows social‑login description incorrectly

3 participants