Skip to content

Conversation

@Yulyaswan
Copy link
Collaborator

Request:
Screenshot 2025-09-29 at 11 55 01

Replaced the custom drag-and-drop view implementation for the "Select Entries" (Pro) variable
Now it uses the built-in Relationship field instead

@Yulyaswan Yulyaswan added this to the 7.5.x milestone Sep 29, 2025
@robinsowell
Copy link
Contributor

So I don't know if I missed something setting it up or not, but when on the Relationship version, it's not displaying more than one select after saving.

Before upgrading, I had 2 options selected. After upgrading, only 1 was showing. But the data were correct in the field. If I save with only 1 showing, then it updates to only that 1 saved. Which does make sense.

I go do it and select multiples- I save- it properly updates to a pipe delimited list in the database- but still only 1 shows.

If I have it in multi-select mode, it's working as expected.

Display and db data below:

Screenshot 2025-10-08 at 12 59 10 PM

It's not impossible that I didn't replace a file or something- but can you double check that multiples are displaying properly?

Copy link
Contributor

@robinsowell robinsowell left a comment

Choose a reason for hiding this comment

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

Not sure this is working- can you double check the display of multiples? More detail in comments.

@Yulyaswan Yulyaswan requested a review from robinsowell October 10, 2025 10:38
Copy link
Contributor

@robinsowell robinsowell left a comment

Choose a reason for hiding this comment

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

This got it working for me. I did find 1 tiny thing:

Screenshot 2025-10-23 at 5 21 22 PM

I'm not sure why that one has the unparsed markup in the message- I checked on relationship in a regular entry and it doesn't.

@TomJaeger TomJaeger merged commit a5a6c6d into 7.dev Oct 24, 2025
24 checks passed
@TomJaeger TomJaeger deleted the feature/7.x/provar-changeVarTo-relationship-field branch October 24, 2025 20:14
@TomJaeger TomJaeger modified the milestones: 7.5.x, 7.5.17 Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants