-
Notifications
You must be signed in to change notification settings - Fork 147
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
docs(frontend): 2.7x doc review #917
Conversation
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.
Looks good for me for common-errors. Having a look to the rest
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.
Some comments. To me, it's approved for "common_errors" and for the rest, I'll let @antoniupop and @aPere3 comment on their parts.
When we're ok with the text, @yuxizama, don't worry we can fix the CI |
Hey guys, Thanks for the comments, they have been all addressed. Please have a look and we can move on to merge this PR. |
11ade81
to
5216002
Compare
No description provided.