Skip to content

Fixes typos in SWODLR access method and tooltip text - #2099

Closed
BrUn3y wants to merge 2 commits into
nasa:mainfrom
BrUn3y:fix/swodlr-typos
Closed

BrUn3y wants to merge 2 commits into
nasa:mainfrom
BrUn3y:fix/swodlr-typos

Conversation

@BrUn3y

@BrUn3y BrUn3y commented Oct 12, 2026

Copy link
Copy Markdown

Overview

What is the feature?

Fixes spelling, capitalization and spacing errors in user-facing SWODLR text. Closes #2098.

What is the Solution?

  • AccessMethod.jsx: avaliable → available in the "Generate with SWODLR" access method details
  • swodlrConstants.js tooltips:
    • GranuleExtent: stich → stitch
    • MGRS: L@_HR_Raster → L2_HR_Raster, each other , → each other,
    • UTM: universal → Universal, each other , → each other,, (+/- 1 zone)to → (+/- 1 zone) to

Text-only changes, no logic changes.

I don't have an EDSC ticket number, so the commit message doesn't use the EDSC-XXXX: prefix. I'm happy to reword it if a maintainer assigns one.

What areas of the application does this impact?

The SWODLR access method in the project panel and the SWODLR form tooltips.

Testing

Reproduction steps

  • Environment for testing: local
  • Collection to test with: any SWOT collection that supports SWODLR
  1. Add the collection to a project and open the project panel.
  2. Check the "Generate with SWODLR" access method details text.
  3. Select it and hover the Granule Extent, MGRS Band Adjust and UTM Zone Adjust tooltips.

Checklist

  • I have added automated tests that prove my fix is effective or that my feature works (N/A, text only; no tests assert on these strings)
  • New and existing unit tests pass locally with my changes (AccessMethod and SwodlrForm suites, 73 tests, Node 22)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas (N/A)
  • I have made corresponding changes to the documentation (N/A)
  • I have run npm audit fix and made note of any changes in this PR (N/A, no dependency changes)
  • My changes generate no new warnings

@BrUn3y

BrUn3y commented Nov 3, 2026

Copy link
Copy Markdown
Author

Hi @eudoroolivares2016, could you please review this PR and approve it if everything looks good? I saw you've worked on the SWODLR form, so I thought you'd be a good person to ask. This only fixes spelling, capitalization, and spacing in the access method text and tooltips; there are no logic changes. The AccessMethod and SwodlrForm test suites passed locally (73 tests, Node 22). Thanks!

@BrUn3y

BrUn3y commented Nov 11, 2026

Copy link
Copy Markdown
Author

Hi team, could someone please approve the pending workflow runs for this PR? The CI (#4930) and Compare Bundle Size (#1651) runs for commit 0cce19a expired after waiting 30 days for approval ("This workflow run required approval but was not approved before it expired"), and the newer runs from Nov 2 (CI #4963, Compare Bundle Size #1679) are still awaiting approval. No jobs ran, so this is not a test failure — just the approval step. Thanks!

@mandyparson

Copy link
Copy Markdown
Member

Hello! And thank you for your contribution. We've recently updated our guidelines here to help contributors like yourself get your work considered by our team.

I can see you already mentioned that you were missing an EDSC-XXXX number, and that's going to be my advice here. Submit your suggestion into github issues and wait to be assigned an EDSC-XXXX number at that time. Again, we really appreciate you reaching out and I've made a note about this in my notes as well. Let me know if you have any questions about on contribution process in the future.

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.

Fix typos in SWODLR access method and tooltip text

2 participants