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

[FIX] developer: rewrite a bunch of outdated info in JS reference #9180

Closed
wants to merge 1 commit into from

Conversation

sdegueldre
Copy link
Contributor

Previously, a lot of information on the JavaScript reference page was outdated.

This commit rewrites a bunch of sections to bring them up to date and removes others that are no longer relevant. While a more profound rework of the JS documentation is still needed, this commit at least reduces the amount of outdated information significantly in the meantime.

@robodoo
Copy link
Collaborator

robodoo commented May 6, 2024

Copy link
Collaborator

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

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

LGTM 👍

I only commented on the form and used change suggestions so that you can directly apply them through GitHub.

@robodoo delegate+

Previously, a lot of information on the JavaScript reference page was
outdated.

This commit rewrites a bunch of sections to bring them up to date and
removes others that are no longer relevant. While a more profound rework
of the JS documentation is still needed, this commit at least reduces
the amount of outdated information significantly in the meantime.
@sdegueldre sdegueldre force-pushed the 17.0-rewrite-js-reference-fix-sad branch from 975ebf2 to d698107 Compare May 6, 2024 13:33
@sdegueldre
Copy link
Contributor Author

Thanks for taking the time to review this and sorry for all the typos. Either I need more coffee or something is wrong with my keyboard 😅 Applied all comments and squashed.

@robodoo r+

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.

The link is broken and redirects to a 404 page from version 14 to version 17
3 participants