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

Added Demo for SmartMultiInput feature #463

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Conversation

mzinnoecker
Copy link
Contributor

See PR 1630

@oblomov-dev oblomov-dev merged commit c56ff1a into abap2UI5:main Dec 10, 2024
4 checks passed
@oblomov-dev
Copy link
Member

everything merged, i think it should work now. thank you for the PR!

Unfortunately it seems that i can not test it on ABAP dev edition:
image

do you have some screenshots or a link to the documentation that i can see what it is about? maybe i can find a service in the dev edition for that as well.

@mzinnoecker
Copy link
Contributor Author

Thanks!
Yes i will take some screenshots tomorrow :)
Essentially it enables the usage of SmartMultiInput. If annotated correctly (Consumption.ValueHelpDefinition) we have to also link the backend annotations in for the ui5 odata model, thats why i added the parameter switchDefaultModelAnnoURI. It is required for the smartMultiInput to render the table/auto suggestions based on the backend annotations.

In short: will take some screenshots tomorrow :)

@mzinnoecker
Copy link
Contributor Author

hey, as promised:

Service Binding
Product Type
Selection
Auto completion
range content in backend

@oblomov-dev
Copy link
Member

thank you for the screenshot, ah its this feature, always used it automatically with Fiori Elements, but never accessed it manually in UI5 freestyle. Very nice, thank you for figuring it out and adding it to abap2UI5.

adjusted your sample to the demo travel odata service which is available on dev editions:
Eclipse 2024-12-11 13 10 00
Google Chrome 2024-12-11 13 10 19

And renamed for consistency to:
switch_default_model_anno_uri

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants