Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: organize project structure #28

Merged
merged 2 commits into from Mar 5, 2019

Conversation

Projects
None yet
2 participants
@marcgarreau
Copy link
Collaborator

marcgarreau commented Mar 4, 2019

What does it do?

Fairly big overhaul of directory layout. See screenshot.

Any helpful background information?

  • It's not a big app; removed containers directory in favor of tracking each tab of the app in a directory within components. I think its easier to reason about where things are located this way.
  • Filed away shared components and popups related stuff. The popups aren't relevant yet anyway.
  • This will rekt your afternoon with merge conflicts. Sorry about that.

Relevant screenshots?

screen shot 2019-03-04 at 4 18 23 pm

marcgarreau and others added some commits Mar 4, 2019

@ryanio

ryanio approved these changes Mar 4, 2019

Copy link
Collaborator

ryanio left a comment

looks great!

@ryanio ryanio merged commit acd5aa4 into master Mar 5, 2019

2 checks passed

Semantic Pull Request ready to be squashed
Details
ci/circleci: build Your tests passed on CircleCI!
Details

@marcgarreau marcgarreau deleted the shuffle branch Mar 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.