Skip to content

Update README.md to emphasize that CustomJS only supports single class definition per file#104

Merged
saml-dev merged 1 commit into
saml-dev:masterfrom
jeansordes:master
Oct 4, 2025
Merged

Update README.md to emphasize that CustomJS only supports single class definition per file#104
saml-dev merged 1 commit into
saml-dev:masterfrom
jeansordes:master

Conversation

@jeansordes

Copy link
Copy Markdown
Contributor

I just tried using CustomJS … and couldn't figure out how to make it work, until I went in the issues sections and found that the explanation was (as often), that we collectively tend to skim the doc and skip the part where CustomJS only supports single class definition per file

I've added examples of incorrect and correct usage to clarify the requirements for file structure

Hope it will help other users

(And by the way, thanks for maintaining the plugin 🙏)

…s definitions per file. Added examples of incorrect and correct usage to clarify the requirements for file structure.
@saml-dev

saml-dev commented Oct 4, 2025

Copy link
Copy Markdown
Owner

I appreciate the attempt, hopefully it makes a difference. Many people in open source create an issue before reading the documentation, so there's only so much you can do 🤷🏼‍♂️

@saml-dev saml-dev merged commit d21ab2f into saml-dev:master Oct 4, 2025
2 checks passed
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.

2 participants