-
Notifications
You must be signed in to change notification settings - Fork 655
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
link infowindows with cartodb_id #8127
Conversation
not sure if I'm the best to review this one since I don't know so much about the queryModel |
return m.get('name') === 'cartodb_id'; | ||
}); | ||
|
||
if (cartdb_id) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, what would/should happen if isEmptyTemplate
and there's no cartodb_id
column in the results? This scenario could happen if the node results is not available yet for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it would apply the changes in the map infowindow from the attributes passed in the change event
Looks legit, added a small comment re: a detail that this change introduce I wasn't sure about though. Also, while I was explained offline why this change was necessary I'd prefer to have a test case that outlines and asserts the scenario. |
yeah, tests on their way, right now they are broken :/ |
🚀 |
CR @javisantana