Skip to content

Fixed Podcast link in Portal Account Settings#27181

Merged
9larsons merged 2 commits intomainfrom
fix-view-link-portal
Apr 6, 2026
Merged

Fixed Podcast link in Portal Account Settings#27181
9larsons merged 2 commits intomainfrom
fix-view-link-portal

Conversation

@9larsons
Copy link
Copy Markdown
Contributor

@9larsons 9larsons commented Apr 6, 2026

ref 35cbbbf

This was previously updated to remove the target="_blank" so that it opens in the same tab, however because Portal is in an iframe it needs to use the parent context in order to move navigation appropriately.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 6, 2026

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d3397088-e940-4088-a5d9-addbf7071a7a

📥 Commits

Reviewing files that changed from the base of the PR and between bfe1c42 and e7a8800.

📒 Files selected for processing (3)
  • apps/portal/package.json
  • apps/portal/src/components/pages/AccountHomePage/components/transistor-podcasts-action.js
  • apps/portal/test/unit/components/pages/AccountHomePage/transistor-podcasts-action.test.js

Walkthrough

This patch release includes a version bump from 2.67.8 to 2.67.9. The Transistor podcasts link in the AccountHomePage component now includes the target="_parent" attribute, which specifies the navigation target for the anchor element. The test suite has been updated with a new assertion to verify that the link element includes both target="_parent" and rel="noopener noreferrer" attributes alongside the existing href validation.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fixed Podcast link in Portal Account Settings' accurately describes the main change: fixing the Transistor podcast link behavior by adding target='_parent' to open in parent context.
Description check ✅ Passed The description is directly related to the changeset, explaining the context and rationale for the fix to the Transistor podcast link when Portal is in an iframe.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-view-link-portal

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.

@9larsons 9larsons merged commit e7481e9 into main Apr 6, 2026
38 checks passed
@9larsons 9larsons deleted the fix-view-link-portal branch April 6, 2026 16:18
franky19 pushed a commit to franky19/Ghost that referenced this pull request Apr 18, 2026
ref 35cbbbf

This was previously updated to remove the `target="_blank"` so that it
opens in the same tab, however because Portal is in an iframe it needs
to use the parent context in order to move navigation appropriately.
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.

1 participant