Repository navigation
Conversation
|
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! |
|
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! |
|
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. |
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→availablein the "Generate with SWODLR" access method detailsswodlrConstants.jstooltips:stich→stitchL@_HR_Raster→L2_HR_Raster,each other ,→each other,universal→Universal,each other ,→each other,,(+/- 1 zone)to→(+/- 1 zone) toText-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
Checklist
AccessMethodandSwodlrFormsuites, 73 tests, Node 22)npm audit fixand made note of any changes in this PR (N/A, no dependency changes)