Detail Popup small issues: about farmview

[stlim0730]

Is this duplicate? (#67)

[calo1]

I'm not sure . I didnt know what the "error" is in #67 so I was documenting my findings in my own words

[stlim0730]

So, the problem seems that the new dataset we synced has lost attachment (optional image) or changed the structure of it. I think the latter has higher chance.

Simplest measure: I can disable image attachment in the code.
More complete measure: investigate how the structure has changed and let the code work with it.

@calo1, what do you think?

[calo1]

@stlim0730

Let's try to investigate for a little while. Fall back is to disable images before the workshop

@karingoh what is the column name for images supposed to be after your processing script?

[stlim0730]

I'm on it.

[calo1]

the two photo column names are

photo_filename and
photolink

I tested the link in photolink and chrome rendered the image correctly

[stlim0730]

Fixed.

The app hasn't used photolink data, but _attachments because it both has links to the thumbnail and original size image.

The data structure hasn't changed. What's changed was that Carto used to give it to us in serialized JSON string, but I found it is in JSON object containing serialized JSON string today. Simple fix worked.

... but why now, Carto?