Skip to content

New and updated diagrams#345

Merged
leandrumartin merged 2 commits intomainfrom
updated-diagrams
Apr 16, 2026
Merged

New and updated diagrams#345
leandrumartin merged 2 commits intomainfrom
updated-diagrams

Conversation

@leandrumartin
Copy link
Copy Markdown
Collaborator

@leandrumartin leandrumartin commented Apr 13, 2026

Pull Request Summary

A brief description/summary of your PR here. What does it add? Why is it necessary? How was the change made, and how was it tested?

  • Database structure diagram updated with lines to visually show connections between database items. For example, a line is added from the boneset bones array to the bone id field to visually show that the array contains a list of strings corresponding exactly to the IDs of various bones.
  • New backend and overall data flow diagrams are added to show new project architectural direction

Detailed Description

A more detailed description and any additional information.

We will be moving the project away from storing our data in a branch on GitHub. Instead, it will be stored in a folder on the server. This will:

  • remove points of failure, making our project more reliable and less prone to failure
  • make our project more secure by eliminating attacks on GitHub as a way to attack our app
  • allow us to develop faster and easier. If you're working on a feature and it requires that data be updated, you would be able to make the appropriate changes in the same app instead of making a pull request and needing it to be accepted before you can even verify whether the change achieved the result you want.

@leandrumartin
Copy link
Copy Markdown
Collaborator Author

@Joishee05 @Brehana-Naidu Let me know whether you think the diagrams are clear and understandable, and if you have any questions on them or about the project direction.

(As a side note, this project direction should be reflected in our showcase presentation. Hopefully we will be able to move the data from GitHub onto our server by the end of the semester.)

Copy link
Copy Markdown
Collaborator

@Brehana-Naidu Brehana-Naidu left a comment

Choose a reason for hiding this comment

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

It looks good, but I honestly can't see any difference between the old and new overall data flow files.

Copy link
Copy Markdown
Collaborator

@Joishee05 Joishee05 left a comment

Choose a reason for hiding this comment

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

Everything looks good to me!

@leandrumartin leandrumartin merged commit f0ce5b9 into main Apr 16, 2026
5 checks passed
@leandrumartin leandrumartin deleted the updated-diagrams branch April 16, 2026 23:29
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.

3 participants