-
Notifications
You must be signed in to change notification settings - Fork 1
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
[bug ?] Work better with forms #1
Comments
Looking into your code, you implemented the third point already. The field-names for Lat and Long don't show up in the bindings suggestions of the form the geolocation component is in. There is a number input component and your geolocation component in the same form I'm setting a form field with the number input called stopKM And I'm setting two form fields with your geolocation input, called stopLong and stopLat Then I want to use these fields in a query Unfortunately, while stopKM from the number input is being forwarded to the query correctly, stopLong and stopLat just stay at their default value 0 when the query is executed. The two number fields also don't show up in the bindings editor Then on the other hand, during execution, the debugger shows me that the form has received the correct numbers: I'm very new to budibase. I assume this to be an issue with the geolocation component because everything works with the built-in number input component. What do you think @andz-bb ? |
Hey @phischdev, thanks so much for your interest in my plugin and for your detailed investigation. Based on your suggestion, I have implemented configurable validation for the latitude and longitude fields in v1.4.0 of the plugin. I have just pushed an update v1.5.2 that includes the automatic mode that you've suggested. It will request the location after the configured delay. I've also added settings to hide the button or the output. I've been unable to recreate the bindings issues on my end and have been able to send my lat/long values as bindings to a REST query successfully. Could you send me your app export or a demo app so that I can investigate the issues that you're seeing? Any help would be much appreciated! Thanks again for your contributions here, have a great weekend! 😎 |
Hey @andz-bb, Out of interest: What is the use case for delayed auto-location? Regarding my issue/bug:
I'll try to setup a demo app soon Enjoy your week! |
hey @phischdev - glad to hear those bits are working! I had imagined it might be frustrating if the user was immediately asked for location permissions when loading a screen with the component, so I added in the timer as an optional way to delay that popup so that it is maybe less annoying for the end user. I don't know if people will find it useful but I thought it was worth throwing in anyway. If it is unnecessary or confusing I can take it out :) Let me know when you have a small demo ready and I will gladly have a look. Thanks again for your contributions! |
Hey Andrew,
thank you for releasing this component.
Is there any way to make it more compatible with forms?
Best,
Philipp
The text was updated successfully, but these errors were encountered: