Skip to content

Redesign ARO Map#595

Merged
Scr4tch587 merged 11 commits intomainfrom
kai/redesign-ARO-map
Nov 1, 2025
Merged

Redesign ARO Map#595
Scr4tch587 merged 11 commits intomainfrom
kai/redesign-ARO-map

Conversation

@Scr4tch587
Copy link
Contributor

Purpose

Redesigning the ARO map-form to match the UI design from the Orbital figma

New Changes

Explain new changes below in short bullet points.

  • Changed the state handling in the new-request form to use React Query
  • Completely implemented the input-form and map-view based on the design from the figma
  • Added alerts for coordinate validation, submission, and initial load-in (temp solution)
  • Selected coordinates default to the user's current geolocation

Testing

Explain tests that you ran to verify code functionality.

  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested.
  • I have tested this PR on a board if the code will run on a board (Only required for firmware developers).
  • I have tested this PR by running the ARO website (Only required if the code will impact the ARO website).
  • I have tested this PR by running the MCC website (Only required if the code will impact the MCC website).
  • I have included screenshots of the tests performed below.

New-request page after initial loading
image

Alert examples
image
image

Outstanding Changes

If there are non-critical changes (i.e. additional features) that can be made to this feature in the future, indicate them here.

  • Switch alerts to something less obtrusive and more aesthetic (react hot toast looks like a good option)
  • Add endpoints for form submission in the ARO backend

@github-actions
Copy link

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K 47 23h 9m 145
camspec 17 20h 54m 124
kepler452b123 16 6d 6h 38m 64
Yarik-Popov 7 1d 1h 41m 58
Navtajh04 3 11d 5h 50m 49
sunray4 1 12d 21m 9
proprogrammer504 1 8d 6h 19m 5

⚡️ Pull request stats

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

Nice work, I would get one more reviewer on this

camspec
camspec previously approved these changes Oct 29, 2025
Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

lgtm! small change

camspec
camspec previously approved these changes Oct 30, 2025
Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

lgtm, get one more reviewer on this (preferably a member, other leads are busy with comp)

Copy link
Contributor

@c4bae c4bae left a comment

Choose a reason for hiding this comment

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

looks good, just very minor qol change

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

lgtm

@Scr4tch587 Scr4tch587 merged commit c69676c into main Nov 1, 2025
49 checks passed
@Scr4tch587 Scr4tch587 deleted the kai/redesign-ARO-map branch November 1, 2025 05:12
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.

3 participants