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.

Sign up for GitHub

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

Jump to bottom

Deployment diagram #100

Merged
merged 3 commits into from
Apr 12, 2023
Merged

Deployment diagram #100

merged 3 commits into from
Apr 12, 2023

Conversation

Pelayo-Reguera
Copy link
Contributor

@Pelayo-Reguera Pelayo-Reguera commented Apr 11, 2023 •
edited
Loading

A table with a small description of each component will be added in a moment

@Pelayo-Reguera Pelayo-Reguera added the documentation Improvements or additions to documentation label Apr 11, 2023
@Pelayo-Reguera Pelayo-Reguera self-assigned this Apr 11, 2023
uo282892
uo282892 requested changes Apr 11, 2023
View reviewed changes
Copy link
Contributor

@uo282892 uo282892 left a comment

Choose a reason for hiding this comment

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

The database is on the cloud (MongoDB Atlas), and you should draw a connection from the restapi to there.

Then, the POD is provided by Inrupt,so you could do the same drawing as in the database.

The webapp and the restapi are on separate containers inside the Azure machine.

@Pelayo-Reguera Pelayo-Reguera marked this pull request as ready for review April 11, 2023 23:47
@Pelayo-Reguera Pelayo-Reguera linked an issue Apr 11, 2023 that may be closed by this pull request
Section 7 - Deployment view #14
Closed
uo282892
uo282892 reviewed Apr 12, 2023
View reviewed changes
Copy link
Contributor

@uo282892 uo282892 left a comment

Choose a reason for hiding this comment

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

I would put that the connection to the pod is from the webapp, not just the server. The same with the database and the restapi

@Pelayo-Reguera
Copy link
Contributor Author

I would put that the connection to the pod is from the webapp, not just the server. The same with the database and the restapi

Yes, it's true. Later I'll change it

UO283428
UO283428 previously approved these changes Apr 12, 2023
View reviewed changes
@UO283428 UO283428 self-requested a review April 12, 2023 15:35
@UO283428 UO283428 dismissed their stale review April 12, 2023 15:36

I thought Ivan's request was fulfilled.

@GuillermoDylan GuillermoDylan self-requested a review April 12, 2023 20:39
@uo282892 uo282892 merged commit 7c1b214 into develop Apr 12, 2023
@Pelayo-Reguera Pelayo-Reguera deleted the Section_7_update branch April 12, 2023 22:03
@Pelayo-Reguera Pelayo-Reguera mentioned this pull request Apr 12, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GuillermoDylan GuillermoDylan GuillermoDylan approved these changes

@uo282892 uo282892 uo282892 approved these changes

@Davidmc07 Davidmc07 Davidmc07 approved these changes

@UO283428 UO283428 UO283428 approved these changes

Assignees

@Pelayo-Reguera Pelayo-Reguera

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
Prototype + Documentation 1.0
Development

Successfully merging this pull request may close these issues.

Section 7 - Deployment view
5 participants
@Pelayo-Reguera @UO283428 @Davidmc07 @uo282892 @GuillermoDylan