neo4j / neo4j-browser

Neo4j Browser is the general purpose user interface for working with Neo4j. Query, visualize, administrate and monitor the database.
https://neo4j.com
GNU General Public License v3.0
679 stars 345 forks source link

URL param support for populating editor with Cypher does not work #1070

Closed achantavy closed 4 years ago

achantavy commented 4 years ago

Description

I followed the examples here: https://github.com/neo4j/neo4j-browser/pull/956 and cannot get neither Neo4j 3.5.16 Community nor Neo4j 4.0.1 Community to populate the editor with Cypher. The edit textbox remains empty in both Chrome and Safari.

Reproduce Steps

  1. Download of Neo4j 3.5.16 Community or 4.0.1 Community.
  2. Run ~/neo4j-{version}/neo4j console
  3. Go to http://localhost:7474/?cmd=edit&arg=RETURN%201;&arg=RETURN%20rand(); as described in https://github.com/neo4j/neo4j-browser/pull/956
  4. Text field does not fill up with expected text.

Expected Behavior

Text should fill up in the box as shown in #956.

Current Behavior

Chrome and Safari route me to localhost:7474/browser and nothing is displayed in the text box.

Your Environment

software name version
OS OSX 10.14.16
Web browser Chrome 80.0.3987.132, Safari 13.0.5
node.js N/A
npm N/A
yarn N/A
Neo4j Browser 4.0.3 and 4.0.3
Neo4j I tried out 3.5.16 Community and 4.0.1 Community

Additional Information

Thanks for your help!

achantavy commented 4 years ago

cc: @oskarhane

oskarhane commented 4 years ago

What if you add the /browser path @achantavy ? http://localhost:7474/browser/?cmd=edit&arg=RETURN%201;&arg=RETURN%20rand();

jexp commented 4 years ago

Does this work for you with Desktop?

neo4j-desktop://graphapps/neo4j-browser/?cmd=edit&arg=RETURN%201

try this with /browser

http://localhost:7474/browser/?cmd=edit&arg=RETURN%201

jexp commented 4 years ago

ping @achantavy

achantavy commented 4 years ago

Hey @oskarhane, @jexp,

adding browser/ worked, thanks! Maybe you can add in an edit to the first comment of #956 to add in the browser path.

Edit: eh, it's an old PR so maybe not, but this took me way long to find lol.