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

Add some diagram of app project #63

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Add some diagram of app project #63

wants to merge 2 commits into from

Conversation

Magicianred
Copy link

Insert some schemes to start having an overview of the app. To evaluate and improve with developers

@Magicianred Magicianred added documentation Improvements or additions to documentation help wanted Extra attention is needed review developer an approve by a developer labels May 9, 2020
@dcremonini
Copy link

dcremonini commented May 10, 2020

Hi Simone,
thank you for contributing.
Here are my remarks:

  • the documentation is in Italian (everything we publish should be in English - this is true for filenames and contents)
  • in the diagram I see a mix of Android front-end and back-end mixed together. Maybe this repository is not the best one

Later I will also check the content

@Magicianred
Copy link
Author

Thanks @dcremonini for the tips, you're right

these are a first draft to start participating, the architecture is not exactly like the original and in fact I am trying to review it together with a developer who has seen the code and allows me to bring it closer to the real situation

The .drawio files are editable with the app https://app.diagrams.net/ and I invite everyone to make the corrections they believe useful for improvement and to make them more usable for everyone (for example by translating the texts in English)

or if anyone knows of another easy tool for making uml diagrams we can evaluate them

@Magicianred Magicianred added this to In progress in app-android May 11, 2020
@Magicianred Magicianred moved this from In progress to To vote or To take into account in app-android May 11, 2020
@Magicianred Magicianred moved this from To vote or To take into account to To do in app-android May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed review developer an approve by a developer
Projects
app-android
  
To do
Development

Successfully merging this pull request may close these issues.

None yet

2 participants