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

Add fullObjectValue to lookup options #188

Merged
merged 2 commits into from
Jun 26, 2020
Merged

Add fullObjectValue to lookup options #188

merged 2 commits into from
Jun 26, 2020

Conversation

dmitrp
Copy link
Collaborator

@dmitrp dmitrp commented Jun 26, 2020 •
edited
Loading

Added full object option value for elements-ui react Lookup component when fullObjectValue option is set to true

@dmitrp dmitrp requested a review from dmarcautan June 26, 2020 05:45
@dmarcautan
Copy link
Collaborator

  1. IMPORTANT Rebase to latest master (0.11.0) and update version accordingly
  2. Provide description in PR for use cases of your changes (Will be included in release notes after published)

dmarcautan
dmarcautan requested changes Jun 26, 2020
View reviewed changes
Copy link
Collaborator

@dmarcautan dmarcautan left a comment •
edited by dmitrp
Loading

Choose a reason for hiding this comment

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

  • IMPORTANT Rebase to latest master (0.11.0) and update version accordingly
  • Provide description in PR for use cases of your changes (Will be included in release notes after published)

@dmitrp dmitrp force-pushed the f_dpi_LookupFullObjectValueOptions branch from 4cf4ef1 to 3d7e60c Compare June 26, 2020 06:58
@dmitrp dmitrp merged commit 6ea04a5 into master Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmarcautan dmarcautan dmarcautan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dmitrp @dmarcautan