docs: update old Transifex URLs in release notes#7239
docs: update old Transifex URLs in release notes#7239thepastaclaw wants to merge 4 commits intodashpay:developfrom
Conversation
✅ No Merge Conflicts DetectedThis PR currently has no conflicts with other open PRs. |
thepastaclaw
left a comment
There was a problem hiding this comment.
Code Review
Docs-only PR replacing 7 dead Transifex URLs in historical release notes. The old www.transifex.com/projects/p/dash/ links are replaced with app.transifex.com/dash/, which works (302 redirect) but is the organization-level URL rather than the project-specific explore.transifex.com/dash/dash/ used in README.md and doc/translation_process.md. The darkcoin→dash rebrand in v0.11.2 is handled correctly.
Reviewed commit: 454d0af
🟡 1 suggestion(s)
🤖 Prompt for all review comments with AI agents
These findings are from an automated code review. Verify each finding against the current code and only fix it if needed.
In `doc/release-notes/dash/release-notes-0.11.2.md`:
- [SUGGESTION] line 110: Replacement URL points to org page rather than project page used elsewhere in repo
All 7 release notes now link to `https://app.transifex.com/dash/` (organization-level, returns HTTP 302 redirect), while `README.md:96` and `doc/translation_process.md:11,45` consistently use `https://explore.transifex.com/dash/dash/` (project-level, returns HTTP 200 directly). The current links are functional since the redirect lands in the right place, but using `explore.transifex.com/dash/dash/` would be consistent with the rest of the repo and avoid the redirect. Since these are historical release notes that are rarely consulted directly, this is a suggestion rather than a blocker.
|
|
||
| As well as the Bitcoin Core developers and everyone that helped translating on | ||
| [Transifex](https://www.transifex.com/projects/p/darkcoin/). | ||
| [Transifex](https://app.transifex.com/dash/). |
There was a problem hiding this comment.
🟡 Suggestion: Replacement URL points to org page rather than project page used elsewhere in repo
All 7 release notes now link to https://app.transifex.com/dash/ (organization-level, returns HTTP 302 redirect), while README.md:96 and doc/translation_process.md:11,45 consistently use https://explore.transifex.com/dash/dash/ (project-level, returns HTTP 200 directly). The current links are functional since the redirect lands in the right place, but using explore.transifex.com/dash/dash/ would be consistent with the rest of the repo and avoid the redirect. Since these are historical release notes that are rarely consulted directly, this is a suggestion rather than a blocker.
💡 Suggested change
| [Transifex](https://app.transifex.com/dash/). | |
| [Transifex](https://explore.transifex.com/dash/dash/). |
source: ['claude', 'codex']
🤖 Fix this with AI agents
These findings are from an automated code review. Verify each finding against the current code and only fix it if needed.
In `doc/release-notes/dash/release-notes-0.11.2.md`:
- [SUGGESTION] line 110: Replacement URL points to org page rather than project page used elsewhere in repo
All 7 release notes now link to `https://app.transifex.com/dash/` (organization-level, returns HTTP 302 redirect), while `README.md:96` and `doc/translation_process.md:11,45` consistently use `https://explore.transifex.com/dash/dash/` (project-level, returns HTTP 200 directly). The current links are functional since the redirect lands in the right place, but using `explore.transifex.com/dash/dash/` would be consistent with the rest of the repo and avoid the redirect. Since these are historical release notes that are rarely consulted directly, this is a suggestion rather than a blocker.
The old Transifex project URL (www.transifex.com/projects/p/dash/) has been replaced with the current URL (app.transifex.com/dash/) across all release notes that still referenced it.
454d0af to
99723d6
Compare
thepastaclaw
left a comment
There was a problem hiding this comment.
Code Review
The new push addresses the suggestion from the prior review — all 7 release notes now use explore.transifex.com/dash/dash/ (the project-level URL) instead of app.transifex.com/dash/ (org-level URL with redirect). This is consistent with README.md and doc/translation_process.md. Clean incremental, LGTM.
Reviewed commit: 99723d6
Summary
Update the old Transifex project URL (
www.transifex.com/projects/p/dash/) to the current URL (app.transifex.com/dash/) in historical release notes.Changes
7 release notes files updated (v0.11.2 through v0.13.0) — these were the only files still referencing the deprecated URL format. The
darkcoinvariant in v0.11.2 was also updated to point to the current Dash project page.Note:
README.mdanddoc/translation_process.mdalready use the currentexplore.transifex.com/dash/dash/URL and required no changes.Validation
www.transifex.com/projects/p/in the repohttps://app.transifex.com/dash/is the correct current URL