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

Merge with documentation #38

Merged
merged 91 commits into from
Feb 23, 2024
Merged

Merge with documentation #38

merged 91 commits into from
Feb 23, 2024

Conversation

AbelMH1
Copy link

@AbelMH1 AbelMH1 commented Feb 23, 2024

Mergin all the changes from develop into master with the first version of the documentation

ExarcaFidalgo and others added 30 commits January 22, 2024 17:05
This reverts commit dbaea69, reversing
changes made to fa9aec3.
Little fixes into the README.md and deploy.yml to tear-down the containers correctly
Added most of the constraints (Technical/Organizational/Conventions). Some changes/additions may be required.
Titles didn't pop as much as they should. 
Bold font for the constraints.
Cursive for the appropriate words (Acronyms, software terminology... etc.)
Added deployment diagram for the technical context.
Added business context.
Previously, the diagram did not account for the bidirectional nature of communications between User-WebApp and API-WebApp.
uo283642 and others added 25 commits February 22, 2024 17:56
PlantUML annotations didnt work :(
Removed annotations, changed for image. Annotations didn't work :(
Diagram img wasn't cropped correctly
Changes in section 2 and 5 of the documentation.
@AbelMH1 AbelMH1 marked this pull request as ready for review February 23, 2024 07:22
HOTFIX: Deleted local directory from visual studio code
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@alegarman2002 alegarman2002 merged commit 21120a0 into master Feb 23, 2024
2 checks passed
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.

9 participants