feature: async choices parseValue

prestojs / prestojs

https://prestojs.com/

6 stars 1 forks source link

feature: async choices parseValue #173

Closed davecoates closed 2 years ago

davecoates commented 2 years ago

Add parseValue to support converting values (eg. string => number). This allows better matching when input may come from string source but values are integers.
Fix handling of useAsyncValue existingValues to work with complex keys

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
prestojs	✅ Ready (Inspect)	Visit Preview	Aug 25, 2022 at 0:06AM (UTC)

davecoates commented 2 years ago

Yeah needs this change in async_chocies.html:

@@ -52,4 +52,9 @@ new AsyncChoices<{ {{options.label_field}}: string; {{options.value_field }}: {{
     getValue(item) {
         return item.{{ options.value_field }};
     },
+   {% if options.value_type_info == 'number' %}
+     parseValue(value) {
+        return Number(value);
+     },
+   {% endif %}
 })