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

Include option set labels #2

Closed
akaskela opened this issue Dec 7, 2018 · 6 comments

Comments

@akaskela
Copy link

commented Dec 7, 2018

It'd be sweet if you included global and non-global option sets in the search. I'm working with a tool that maps from picklists to a form and having trouble tracking down the source.

@mikefactorial

This comment has been minimized.

Copy link
Owner

commented Dec 11, 2018

Yeah, I have to look at this. I'm excluding fields that have certain Metadata attributes that probably exclude the picklistattribute(name). Thanks Aiden

@mikefactorial mikefactorial self-assigned this Dec 11, 2018

@mikefactorial

This comment has been minimized.

Copy link
Owner

commented Dec 11, 2018

I know I can retrieve those attributes, but I think they throw an exception if you try to search them because they are child attributes. I'd have to use the metadata to determine what value to search based on the text which might be slow. Any thoughts on making that more performant?

@akaskela

This comment has been minimized.

Copy link
Author

commented Dec 11, 2018

You'd have to go through the metadata but make it an option that you have to opt in to for a given search so you don't have to wait unless you want the metadata searched.

@mikefactorial

This comment has been minimized.

Copy link
Owner

commented Dec 11, 2018

Yeah, had to do the same thing for the case sensitive search since it slows it down to do the calculations after retrieving the results. Probably wouldn't add that much more overhead though

@mikefactorial

This comment has been minimized.

Copy link
Owner

commented Jul 16, 2019

This has been resolved in version 1.1.0.

@mikefactorial

This comment has been minimized.

Copy link
Owner

commented Jul 16, 2019

There’s an option to search option set and / or lookup display values now with a warning that searches may be slower as a result

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.