Skip to content

fix: correct spelling errors in UI text#374

Open
Vishal27alpha wants to merge 1 commit intoEAPD-DRB:mainfrom
Vishal27alpha:fix-spelling-errors
Open

fix: correct spelling errors in UI text#374
Vishal27alpha wants to merge 1 commit intoEAPD-DRB:mainfrom
Vishal27alpha:fix-spelling-errors

Conversation

@Vishal27alpha
Copy link
Copy Markdown

Linked issue

  • Closes #
  • Related #

Existing related work reviewed

  • Issues/PRs reviewed: None found after search

Overlap assessment

  • Classification: none
  • Overlapping items: None
  • Why this is not duplicate/superseded:
    This PR fixes minor spelling mistakes in UI text and does not overlap with any existing work.

Why this PR should proceed

  • Improves UI text clarity and professionalism
  • No functional changes involved
  • Small, isolated fix

Summary

  • What changed:
    Fixed spelling mistakes in UI text in AddCase.js and DataFile.js

  • Why:
    To improve readability and correctness of user-facing text

Validation

  • Tests added/updated (or not applicable)
  • Validation steps documented
  • Evidence attached (logs/screenshots/output as relevant)

Documentation

  • Docs updated in this PR (or not applicable)
  • Any setup/workflow changes reflected in repo docs

Scope check

  • No unrelated refactors
  • Implemented from a feature branch
  • Change is deliverable without upstream OSeMOSYS/MUIO dependency
  • Base repo/branch is EAPD-DRB/MUIOGO:main (not upstream)

Exception rationale

This PR qualifies for the narrow docs/typo exception as it only fixes minor spelling mistakes in UI text and does not introduce any functional changes.

Copilot AI review requested due to automatic review settings March 26, 2026 14:08
@github-actions github-actions bot added the needs-intake-fix PR intake structure needs maintainer follow-up label Mar 26, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to improve user-facing UI text clarity by correcting spelling mistakes in controller messages within the WebAPP.

Changes:

  • Fixed “Plese wait...” to “Please wait...” in batch run / cleanup loader messages.
  • Corrected multiple typos in AddCase UI warnings/dialogs (e.g., “neccessary” → “necessary”, “visalisation” → “visualisation”).
  • Updated a scenario deletion alert message text.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
WebAPP/App/Controller/DataFile.js Corrects spelling in loader messages shown during batch run and cleanup actions.
WebAPP/App/Controller/AddCase.js Corrects several typos in warnings/dialogs, but introduces/retains a couple of user-facing wording issues that should be fixed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Vishal27alpha Vishal27alpha force-pushed the fix-spelling-errors branch 2 times, most recently from 140b32d to 02e3f52 Compare March 28, 2026 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-intake-fix PR intake structure needs maintainer follow-up

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants