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

Refactor the directory structure #17

Open
tanius opened this issue Oct 22, 2019 · 0 comments
Open

Refactor the directory structure #17

tanius opened this issue Oct 22, 2019 · 0 comments

Comments

@tanius
Copy link
Member

tanius commented Oct 22, 2019

Putting this here as a reminder, because I know I'd otherwise just get used to it with more exposure.

So: the directory structure could really be logically clearer. Comments on a few of the existing directories:

  • /Images/: Currently contains UI elements and screenshots (= documentation) mixed. Better to be split into two directories. Since the UI elements are not under /app, they possibly are source files of some sort for other images inside /app. If so, the new directory should be named accordingly.

  • /app/images/Partners_logos/: Can be removed, as the software is now no longer just about the OpenCare project context.

  • /app/images/techno/: Rename and place it somewhere outside /app, to be used in the README.md documentation and nowhere else. Not inside the application, as there is no need for that.

  • /app/js/: Would be more logical as /app/scripts/lib/. Even better, they should not be in the source code at all (see Remove the libraries in /app/js/ and rely on packages instead #22).

  • /app/scss/: The whole content of this folder would better be moved into /app/styles/.

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

No branches or pull requests

1 participant