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

[Bug]: Documentation error - Creating system variables via Hop GUI #3943

Closed
dave-csc opened this issue May 16, 2024 · 3 comments
Closed

[Bug]: Documentation error - Creating system variables via Hop GUI #3943

dave-csc opened this issue May 16, 2024 · 3 comments
Assignees
Milestone

Comments

@dave-csc
Copy link

Apache Hop version?

2.8.0

Java version?

openjdk version "21.0.2" 2024-01-16

Operating system

Linux

What happened?

The following documentation page explains how to create system variables in Hop GUI:
https://hop.apache.org/manual/latest/variables.html#_system_properties

However, there is no menu toolbar on the Hop GUI window, neither there is a similar command on the Hop icon menu

Issue Priority

Priority: 3

Issue Component

Component: Documentation, Component: Hop Gui

@bamaer
Copy link
Contributor

bamaer commented May 16, 2024

confirmed, that screenshot is outdated.
FYI, the menu bar can be re-enabled from the configuration perspective -> Look & Feel -> uncheck 'Hide the menu bar', and the system variables have moved to the last tab in the configuration perspective.

@bamaer
Copy link
Contributor

bamaer commented May 16, 2024

.take-issue

@github-actions github-actions bot added this to the 2.9 milestone May 16, 2024
@hansva hansva closed this as completed in 9311fd6 May 19, 2024
@dave-csc
Copy link
Author

Hello, I noticed that documentation has been updated for version 2.9 :)

There is still an error regarding the position in the hop-config.json file where the new variables are saved: in the file there is no section "systemProperties", but new variables are added in the "variables" section like this:

{
  "variables" : [ {
  ...
  }, {
    "name" : "MY_SYSTEM_PROPERTY",
    "value" : "SomeValue",
    "description" : "My new system property/variable"
  } ], 
  ...
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants