-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Deploy Fixes to Production (Mid-Dec) #599
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
Conversation
i found that default authentication route should be `/api/auth/local`
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Signed-off-by: harimkims <harimkims@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
I found that default route user register in v4 is `api/auth/local/register` Co-authored-by: DMehaffy <derrickmehaffy@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Signed-off-by: harimkims <harimkims@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
In the documentation about the connection parameters, the `username` parameter does not actually exist. This is probably a typo and should be replaced by the `user` parameter as per the example given below on the same page. Co-authored-by: DMehaffy <derrickmehaffy@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Update default MySQL Port in example to 3306 Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
- bump MyQL/MariaDB Minimum Version for `JSON` datatype support
There should be two enclosing parenthesis at the end. Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
There are still some references to the v3 auth endpoints (without `/api` at the start) on the Users and Permissions plugin documentation. # What does it do? Updates Strapi endpoints to include `/api` at the start # Why is it needed? This change should be documented; it caused a lot of confusion for me as I was trying to figure out why my authentication was not working on v4. # Related issue(s)/PR(s) A similar thing was fixed for the guides in PR #553 Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> Co-authored-by: DMehaffy <derrickmehaffy@gmail.com>
* lifecycles.js file is not generated automatically I think since version 4, the lifecycles.js file is not created automatically but needs to be created manually, if needed. Postscript Correct me if I'm wrong. You can also rephrase this change if it is relevant and true. Thanks * Update docs/developer-docs/latest/development/backend-customization/models.md Co-authored-by: DMehaffy <derrickmehaffy@gmail.com> * Update docs/developer-docs/latest/development/backend-customization/models.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> Co-authored-by: DMehaffy <derrickmehaffy@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
This pull request is being automatically deployed with Vercel (learn more). documentation – ./docs🔍 Inspect: https://vercel.com/strapijs/documentation/H9cxL1r2dtvix7qurbNKr8889FW3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
See Commit log (changelogs will come eventually, not now)