Merged
Conversation
…t to start API and frontend servers, Update README with correct project setup and run instructions
jacksayshi
approved these changes
Sep 10, 2025
Collaborator
jacksayshi
left a comment
jacksayshi
left a comment
There was a problem hiding this comment.
lgtm. This makes navigating the repo much easier.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
A pull request must contain the following elements.
1. What was Changed
i deleted unused .js files and reduntant html files, to simplify app start up, i added concurrent script to package.json to start API and frontend servers, updated ReadMe file and integrated HTMX for dynamic description loading.
2. Why it was Changed
make the code base easier for developers to work with and avoid conflicts
3. How it was Changed
deleted fetch.js, boneset.js, image_handling.js as well as bone.html as there were no longer needed. then updated the script tag in package.json from root directory so both frontend and backend starts together and directly takes you to web.
i also added the HTMX library to
boneset.html.Updated the
<select>dropdowns inboneset.htmlwithhx-getattributes to fetch descriptions on change.added a new
/api/description/endpoint toserver.jsthat serves HTML fragments.these changes will update the initial data fetch in
api.jsto use an absolute URL, fixing cross-port communication with the backend API.4. Screenshots (if applicable)
not applicable