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

Use as data point location flag does not change with saving #2517

Closed
janagombitova opened this issue Mar 2, 2018 · 4 comments
Closed

Use as data point location flag does not change with saving #2517

janagombitova opened this issue Mar 2, 2018 · 4 comments
Assignees

Comments

@janagombitova
Copy link
Contributor

janagombitova commented Mar 2, 2018

Steps to reproduce:

  1. Create a geolocation question and Use as data point location is by default turned on
  2. Save the question
  3. Open the question again and change the Use as data point location to not be selected. Save
  4. Publish
  5. Reload survey form and check question - use as data point location is turned on.

This happens for any form (monitoring or registration)

See example on UAT1 > _1.9.29.. folder > 1.9.29 All questions (now monitoring)

Change

Ensure that if a user changes the setting Use a data point location, that is is properly saved and published.

Extra notes

  • This issue is only about the Use as data point location flag, as changes to Use as data point name work correctly.
  • This issue might bring up the discussion around the current confusion, that you can set data point location on also in monitoring forms. This is not to be resolved with this issue but later once we start tackling monitoring as a topic.
@kymni kymni self-assigned this Mar 2, 2018
@stellanl
Copy link
Contributor

stellanl commented Mar 2, 2018

Seems to be this helpful code is a little too helpful: question-view.js L156

// when we change the question type to GEO, we turn on the
// localeLocationFLag by default. If we change to something else, we
// turn the flag of.
enableLocaleLocation: function() {
this.set('localeLocationFlag', this.type.get('value') == 'GEO');
}.observes('this.type'),

@janagombitova
Copy link
Contributor Author

@kymni
Copy link
Contributor

kymni commented Mar 5, 2018

The locale flag is currently set to turn on by default when a question is set to type geolocation. To fix the issue, we'll change this so the user has to toggle the option themselves and also stop auto-checking this option in case of a disabled flag

kymni pushed a commit that referenced this issue Mar 5, 2018
stellanl added a commit that referenced this issue Mar 5, 2018
…e-flag

Stop autochecking "Use as data point location" (Connect #2517)
@janagombitova
Copy link
Contributor Author

Use as data point location is not set by default anymore and making changes is now well reflected also in the UI. All good 👍

@janagombitova janagombitova added this to the 1.9.29 C... C... milestone Mar 6, 2018
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

No branches or pull requests

3 participants