Skip to content

Fixed droping QSYS members into IFS folder#2587

Merged
sebjulliand merged 2 commits intomasterfrom
fix/memberDrop
Mar 26, 2025
Merged

Fixed droping QSYS members into IFS folder#2587
sebjulliand merged 2 commits intomasterfrom
fix/memberDrop

Conversation

@sebjulliand
Copy link
Copy Markdown
Member

Changes

For some reason, dropping members into the IFS browser now fails with this message.
image

Getting the members items from the dataTransfer object doesn't give the expected result (anymore?).
However, a URI list is automatically added to the dataTransfer when dragging items, so we now use that to perform the member copy to IFS operation.

How to test this PR

  1. Drag and drop a member from the Object Browser into an IFS folder in the IFS browser
  2. The member must be copied to a streamfile in the target folder.

Checklist

  • have tested my change

Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
@sebjulliand sebjulliand added the bug A confirmed issue when something isn't working as intended label Mar 25, 2025
@sebjulliand sebjulliand self-assigned this Mar 25, 2025
@github-actions
Copy link
Copy Markdown
Contributor

👋 A new build is available for this PR based on d7c1c68.

Copy link
Copy Markdown
Collaborator

@chrjorgensen chrjorgensen left a comment

Choose a reason for hiding this comment

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

@sebjulliand Good catch! I was just caught by this and happy to see your fix was already available. Works as expected - as usual. 👏

@sebjulliand
Copy link
Copy Markdown
Member Author

Thanks @chrjorgensen !
I still don't understand what happened to VS Code...this used to work just fine.

@sebjulliand sebjulliand merged commit dc950cf into master Mar 26, 2025
3 checks passed
@sebjulliand sebjulliand deleted the fix/memberDrop branch March 26, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A confirmed issue when something isn't working as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants