Skip to content

Document jsoneditor extension requirement for ReactFlow#39

Merged
philippjfr merged 13 commits intomainfrom
copilot/fix-jsoneditor-extension-warning
Feb 11, 2026
Merged

Document jsoneditor extension requirement for ReactFlow#39
philippjfr merged 13 commits intomainfrom
copilot/fix-jsoneditor-extension-warning

Conversation

Copy link
Contributor

Copilot AI commented Feb 9, 2026

Fixed UI Test Extension Loading

Updated tests/ui/test_ui.py to call pn.extension("jsoneditor") instead of plain pn.extension(). This ensures the jsoneditor extension is loaded before creating ReactFlow instances in tests, preventing the warning:

WARNING:param.main: pn.extension was initialized but 'jsoneditor' extension was not loaded

Changes

  • tests/ui/test_ui.py: Changed line 13 from pn.extension() to pn.extension("jsoneditor")

This aligns the test file with the documentation updates and ensures consistent extension loading across the codebase.

Original prompt

This section details on the original issue you should resolve

<issue_title>WARNING:param.main: pn.extension was initialized but 'jsoneditor' extension was not loaded</issue_title>
<issue_description>When I run the quick start example https://panel-extensions.github.io/panel-reactflow/#quickstart I see

WARNING:param.main: pn.extension was initialized but 'jsoneditor' extension was not loaded. Since the application is still launching the extension was loaded automatically but we strongly recommend you load the extension explicitly with the following argument(s):

pn.extension('jsoneditor')

Please make sure the quick start example does not raise warnings.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 4 commits February 9, 2026 06:08
…ow.__init__

Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix warning for uninitialized jsoneditor extension Auto-load jsoneditor extension in ReactFlow Feb 9, 2026
Copilot AI requested a review from MarcSkovMadsen February 9, 2026 06:13
Copy link
Contributor

@MarcSkovMadsen MarcSkovMadsen left a comment

Choose a reason for hiding this comment

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

Please note I've reviewed and tested. The copilot proposed import of "jsoneditor" does not work. And I cannot find any other working code.

Please instead document in docs that the user needs to run pn.extension("jsoneditor") when the JSONEditor is available for usage. Please also update any relevant code examples in the documentation.

@MarcSkovMadsen
Copy link
Contributor

@copilot. Please rerun the failing ui tests. Fix if there are errors.

Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 9, 2026

Please note I've reviewed and tested. The copilot proposed import of "jsoneditor" does not work. And I cannot find any other working code.

Please instead document in docs that the user needs to run `...

I've updated the documentation as requested. The auto-loading approach has been removed (you already reverted it in d86ae04), and I've now:

  1. Added prominent note boxes in quickstart.md and index.md explaining that pn.extension("jsoneditor") is required
  2. Updated all code examples in README.md, docs/index.md, and docs/quickstart.md to show the correct extension call
  3. Explained that it must be loaded early in the application before creating ReactFlow instances

Changes committed in 4a38089.

Copilot AI changed the title Auto-load jsoneditor extension in ReactFlow Document jsoneditor extension requirement for ReactFlow Feb 9, 2026
Copilot AI requested a review from MarcSkovMadsen February 9, 2026 09:40
Copy link
Contributor

@MarcSkovMadsen MarcSkovMadsen left a comment

Choose a reason for hiding this comment

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

I'm happy.

@MarcSkovMadsen MarcSkovMadsen marked this pull request as ready for review February 9, 2026 09:41
Co-authored-by: MarcSkovMadsen <42288570+MarcSkovMadsen@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 9, 2026

@copilot. Please rerun the failing ui tests. Fix if there are errors.

I've fixed the UI test file to load the jsoneditor extension. The test was using pn.extension() instead of pn.extension("jsoneditor"), which would cause the same warning we're trying to prevent.

Changed in commit 73f2bf1:

  • tests/ui/test_ui.py line 13: pn.extension()pn.extension("jsoneditor")

The UI tests should now run without the jsoneditor extension warning when the CI rebuilds.

@philippjfr philippjfr merged commit 2eecccf into main Feb 11, 2026
5 of 15 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.

WARNING:param.main: pn.extension was initialized but 'jsoneditor' extension was not loaded

3 participants