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.

Sign up for GitHub

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

Jump to bottom

[Bug]: File separator hardcoded in HopVfsFileDialog #3614

Closed
nadment opened this issue Feb 7, 2024 · 0 comments · Fixed by #3615
Closed

[Bug]: File separator hardcoded in HopVfsFileDialog #3614

nadment opened this issue Feb 7, 2024 · 0 comments · Fixed by #3615
Assignees
@nadment
Labels
bug Hop Gui P3 Nice to have
Milestone
2.8

Comments

@nadment
Copy link
Contributor

nadment commented Feb 7, 2024 •
edited
Loading

Apache Hop version?

2.8.0-SNAPSHOT

Java version?

Java 11

Operating system

Windows

What happened?

Const.FILE_SEPARATOR should be used instead of "/".

C:\Hop\config\projects\default/test-file.hwf

Issue Priority

Priority: 3

Issue Component

Component: Hop Gui
@nadment nadment added this to the 2.8 milestone Feb 7, 2024
@nadment nadment self-assigned this Feb 7, 2024
@github-actions github-actions bot added the P3 Nice to have label Feb 7, 2024
nadment added a commit to nadment/hop that referenced this issue Feb 7, 2024
@nadment
File separator hard coded in HopVfsFileDialog apache#3614
f29546c
@nadment nadment linked a pull request Feb 7, 2024 that will close this issue
File separator hard coded in HopVfsFileDialog #3614 #3615
Merged
usbrandon added a commit that referenced this issue Feb 10, 2024
@usbrandon
Merge pull request #3615 from nadment/3614
2e59a9d 
File separator hard coded in HopVfsFileDialog #3614
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nadment nadment

Labels
bug Hop Gui P3 Nice to have
Projects
None yet
Milestone
2.8
Development

Successfully merging a pull request may close this issue.

File separator hard coded in HopVfsFileDialog #3614 nadment/hop
1 participant
@nadment