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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摑 Update frontend README.md instructions about how to remove the frontend #1121

Closed
wants to merge 1 commit into from

Conversation

johnmathews
Copy link

Currently the instructions in frontend/README.md about how to run the app without a frontend are not complete.

This PR adds 1 step into the frontend/README.md instructions so that the following endpoints will still work if the frontend service is removed:

@deivit24
Copy link

Just deleted a message. Wasn't able to get the docs and api to work but now I did. It was a setup issue with my m2.

@alejsdev alejsdev changed the title update frontend readme instructions about how to remove the frontend 馃摑 Update frontend README.md instructions about how to remove the frontend Apr 3, 2024
@alejsdev alejsdev added the docs label Apr 3, 2024
@walreyes
Copy link

walreyes commented May 1, 2024

thanks for this!

@tiangolo
Copy link
Owner

Thank you! I just updated the Docker Compose files here: #1208

So that it's no longer necessary to add more instructions. 馃

Given that, I'll close this one now, but thanks for bringing this to our attention! 馃嵃

@tiangolo tiangolo closed this May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants