Skip to content

fix/rearrange-file-structure - #51

Merged
elysseaa merged 3 commits into
mainfrom
folder-restructure
Jul 26, 2026
Merged

fix/rearrange-file-structure#51
elysseaa merged 3 commits into
mainfrom
folder-restructure

Conversation

@jvogt23

@jvogt23 jvogt23 commented Jul 25, 2026

Copy link
Copy Markdown
Collaborator

Description

Brief description of changes.
Folders restructured to the following:

  • src
    • Api
    • App
    • Components
    • constants
    • Themes
    • types
    • utils
  • public

These folder restructures are accompanied by minor changes to the files themselves, with some types and constants moved to match the new schema.
Closes #(issue number) - No issue attached

Type of Change

  • Bug fix
  • New feature
  • Refactoring
  • Other (describe):

Changes Made

  • Change 1: File structure moved under src/
  • Change 2: Files updated to move constants and commonly used type defs into constants/ and types/

Testing

How did you test these changes?
Ran application on both an iPhone 10R and a Samsung Galaxy S24 under the apiary-test environment. Scanned attendance. Checked that Settings screen populated correctly. Checked links under Settings such as the Make-A-Wish link, OSS licenses, and Privacy Policy. Logged out.

Metric of test was whether or not these actions successfully executed.

Checklist

  • Linter passes with no errors
  • Changes tested locally
  • No new warnings or errors

@jvogt23
jvogt23 requested a review from elysseaa July 25, 2026 04:12

@elysseaa elysseaa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, PR also seems to include dependency updates with changes to a few iOS/npm config files. More restructuring can be done with the baseline changes made here. Testing app seems to work.

Follow up: I think we could rename the folder names to make casing consistent (my proposed structure didn't exactly do that). But I didn't want to block this PR based on that.

@elysseaa
elysseaa merged commit 3511ed5 into main Jul 26, 2026
1 check passed
@elysseaa
elysseaa deleted the folder-restructure branch July 26, 2026 20:30
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.

2 participants