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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sails-js #167

Merged
merged 13 commits into from
Oct 3, 2024
Merged

Add sails-js #167

merged 13 commits into from
Oct 3, 2024

Conversation

wentelteefje
Copy link
Contributor

This PR adds documentation for sails-js to the Wiki while also re-arranging some pages to increase UX.

Copy link
Contributor

@AndrePanin AndrePanin left a comment

Choose a reason for hiding this comment

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

Awesome! Just updated chapters ordering.

docs/sails-js/sails-js.md Outdated Show resolved Hide resolved
@nikitayutanov
Copy link
Contributor

nikitayutanov commented Sep 25, 2024

What about the Sails functionality in @gear-js/react-hooks? Current README briefly covers it, but I believe frontend devs would benefit from more detailed instructions.

Like Dmitry said, it's CLI generation is not ready yet, but devs can still utilize the hooks manually.

Since the core functionality is based on the generated lib.ts, there's no need to repeat information mentioned in the Overview and sails-js-cli sections. We can just emphasize the slight adjustments that these hooks offer for specific React and general frontend workflows.

Feel free to DM me if you will have any questions regarding it's usage.

docs/sails-js/react-hooks.md Outdated Show resolved Hide resolved
docs/sails-js/react-hooks.md Outdated Show resolved Hide resolved
@AndrePanin AndrePanin merged commit d864a5d into master Oct 3, 2024
1 check passed
@AndrePanin AndrePanin deleted the wf/add-sailsjs-pages branch October 3, 2024 11:15
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.

4 participants