Skip to content

Conversation

jhoover4
Copy link

@jhoover4 jhoover4 commented Jun 7, 2023

Moving things from my-app to root of project.

We're doing this because a proper app setup for a single react app in a repo should have src as a top level folder and package.json in the root.

@ProsperousHeart ProsperousHeart self-requested a review June 8, 2023 00:00
@ProsperousHeart ProsperousHeart added bug / defect A bug was found in a branch and needs to be addressed. documentation Update to documentation good first issue requirements labels Jun 8, 2023
Copy link
Member

@ProsperousHeart ProsperousHeart left a comment

Choose a reason for hiding this comment

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

Unable to approve, since running this branch causes an error:
image

you are missing the entire public folder

@ProsperousHeart
Copy link
Member

To avoid the back and forth and to ensure a clean branch for volunteers moving forward (plus provide other updates needed) we will now be closing this PR to #53. Once it is merged, one of the core admin team members will comment on #49 so you can pull updates from the v1-MVP branch and push back up to #49.

Thank you for your patience and insight.

@jhoover4 jhoover4 deleted the move_src branch June 12, 2023 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug / defect A bug was found in a branch and needs to be addressed. documentation Update to documentation good first issue Not Approved - Changes Requested requirements
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants