Skip to content

Update the apple totp app link.#10097

Open
Malith-19 wants to merge 1 commit intowso2:masterfrom
Malith-19:update-apple-totp-app-link
Open

Update the apple totp app link.#10097
Malith-19 wants to merge 1 commit intowso2:masterfrom
Malith-19:update-apple-totp-app-link

Conversation

@Malith-19
Copy link
Copy Markdown
Contributor

Purpose

  • $subject to replace the outdated apple appstore link.

Related Issues

Related PRs

  • N/A

Checklist

  • e2e cypress tests locally verified. (for internal contributers)
  • Manual test round performed and verified.
  • UX/UI review done on the final implementation.
  • Documentation provided. (Add links if there are any)
  • Relevant backend changes deployed and verified
  • Unit tests provided. (Add links if there are any)
  • Integration tests provided. (Add links if there are any)

Security checks

Developer Checklist (Mandatory)

  • Complete the Developer Checklist in the related product-is issue to track any behavioral change or migration impact.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: e32adfd8-b779-4074-81da-7ec90c6daf3b

📥 Commits

Reviewing files that changed from the base of the PR and between 3980dc4 and 3c939da.

📒 Files selected for processing (3)
  • .changeset/hip-singers-mix.md
  • apps/myaccount/src/components/multi-factor-authentication/authenticators/totp-authenticator.tsx
  • identity-apps-core/apps/authentication-portal/src/main/webapp/enableTOTP.jsp

📝 Walkthrough

Walkthrough

The PR updates the Apple App Store download links in TOTP authenticator components across the codebase. It replaces the Apple generic search URL with a direct App Store iPhone search URL targeted at TOTP applications, reflected in both the MyAccount component and authentication portal.

Changes

Cohort / File(s) Summary
Release Configuration
.changeset/hip-singers-mix.md
Declares a patch release bump for @wso2is/identity-apps-core and @wso2is/myaccount packages.
TOTP App Links
apps/myaccount/src/components/multi-factor-authentication/authenticators/totp-authenticator.tsx, identity-apps-core/apps/authentication-portal/src/main/webapp/enableTOTP.jsp
Updates Apple App Store download links in TOTP UI tooltip and warning messages from https://www.apple.com/us/search/totp?src=globalnav to https://apps.apple.com/us/iphone/search?term=totp.
🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change—updating the Apple TOTP app link—which aligns with all three modified files.
Description check ✅ Passed The description includes a Purpose section addressing the link update and references the related issue #27477. However, the checklist items are left unchecked and the Developer Checklist for behavioral change, migration impact, and new configuration remains incomplete.
Linked Issues check ✅ Passed The PR successfully implements the suggested improvement from issue #27477, updating the Apple URL from the navigation search to an App Store search link across all relevant files.
Out of Scope Changes check ✅ Passed All changes are directly related to updating the Apple TOTP app link as specified in issue #27477. No unrelated code modifications or out-of-scope changes are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Changeset Required ✅ Passed The pull request includes a valid changeset file declaring patch-level version bumps for affected packages with appropriate description.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.05%. Comparing base (9138111) to head (3c939da).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10097   +/-   ##
=======================================
  Coverage   56.05%   56.05%           
=======================================
  Files          42       42           
  Lines        1024     1024           
  Branches      231      231           
=======================================
  Hits          574      574           
  Misses        450      450           
Flag Coverage Δ
@wso2is/core 56.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TOTP authenticator app Link for apple is directs to unrelated web page

1 participant