Skip to content

ERA-13544: update the Ecoscope Downloader link in the sidebar to point to the migration guide that applies post-Downloader sunset (June 30, 2026).#1633

Merged
enlivn merged 2 commits into
developfrom
rr/era-13544-update-ecoscope-downloader-sidebar-link-to-point-to-migration-guide
Jul 2, 2026
Merged

ERA-13544: update the Ecoscope Downloader link in the sidebar to point to the migration guide that applies post-Downloader sunset (June 30, 2026).#1633
enlivn merged 2 commits into
developfrom
rr/era-13544-update-ecoscope-downloader-sidebar-link-to-point-to-migration-guide

Conversation

@enlivn

@enlivn enlivn commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Updates the Downloader link in the sidebar to point to the migration guide that applies post-Downloader sunset (June 30, 2026).

Closes ERA-13544.

Evidence

N/A.

Relevant link(s)

Notes

Just a simple URL tweak.

…igration guide that applies post-Downloader sunset (June 30, 2026).
@enlivn enlivn changed the title ERA-13544 update the Downloader link in the sidebar to point to the migration guide that applies post-Downloader sunset (June 30, 2026). ERA-13544: update the Ecoscope Downloader link in the sidebar to point to the migration guide that applies post-Downloader sunset (June 30, 2026). Jul 2, 2026

@luixlive luixlive left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a small nit if you have 2 minutes to do the swap. It's a non-blocker, this is good to go 👍

Comment thread src/GlobalMenuDrawer/index.js Outdated
@@ -345,7 +345,7 @@ const GlobalMenuDrawer = () => {
<li>
<a
aria-label={t('ecoscopeDownloaderLinkAriaLabel')}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: update the i18n keys to ecoscopeDownloaderMigrationGuideLink and ecoscopeDownloaderMigrationGuideLinkAriaLabel. Those are long keys, but consistent with the others 😄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@luixlive sure. lemme make that change!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Ray, not sure if you want to update the translation text itself, or if we are good keeping "Downloader".
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@luixlive updated 😉

@enlivn enlivn added the deploy label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

❌ PR Environment Deploy Failed

App Sync Health Image
pr-web-rr-era-13544-update-ecoscope-downloader-sidebar-link-to-point-to-migration-guide ⚠️ Unknown ❌ Unknown 7f25f73f42f31b56f059b76ffc71229015e9f6dc

View in ArgoCD

@enlivn enlivn removed the deploy label Jul 2, 2026
@enlivn enlivn merged commit 025fb8e into develop Jul 2, 2026
6 of 7 checks passed
@enlivn enlivn deleted the rr/era-13544-update-ecoscope-downloader-sidebar-link-to-point-to-migration-guide branch July 2, 2026 21:09
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.

2 participants