Skip to content

147 refactor school loc editor#199

Open
shaynesidman wants to merge 2 commits intomainfrom
147-refactor-school-loc-editor
Open

147 refactor school loc editor#199
shaynesidman wants to merge 2 commits intomainfrom
147-refactor-school-loc-editor

Conversation

@shaynesidman
Copy link
Copy Markdown
Collaborator

Issue

Closes #147

Who worked on this sprint/bug?

Shayne

Who did what on this sprint/bug?

Features Implemented

  • Refactored code for school location editor and used it on both the school profile page and the settings page
  • Modified db query for selecting a list of schools to be ordered alphabetically

New files created

Existing files modified

Acceptance Criteria

Testing: how did you test?

Features Not Implemented/Incomplete

  • Did not include the map placing used in the upload flow in the componentization because I feel like it is a bit separate from this one (since schools already have locations in this component).

Bugs Discovered

Screenshots:

Tag Dan and Shayne

@danglorioso @shaynesidman

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mhd Ready Ready Preview, Comment Apr 14, 2026 8:23pm

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.

Refactor: Consolidate MapPicker Components

1 participant