jasonLaster / abuses

4 stars 3 forks source link

Support image types #20

Open jasonLaster opened 4 years ago

jasonLaster commented 4 years ago

The videos.json JSON should have records that link to images instead of youtube.

When that is the case, we should show an image

brwinkle commented 4 years ago

There appears to be some issue with the data. For example, this record:

{
    "id": 32.1,
    "city": "Seattle",
    "state": "WA",
    "tweet": "https://twitter.com/greg_doucette/status/1269637258027642882",
    "file": "Greg Doucette #32.1 - Seattle WA.png",
    "youtube": "",
    "text": "More on 32 in Seattle WA. You'll see more of this in the days ahead. Police will review video footage, get geofence search warrants, etc to charge literally everyone with everything they can. It's all they can do to try and change the narrative"
  },

corresponds to a tweet that links another tweet and does not actually contain an image. There are 69 of these.

This appears to be the only legitimate entry:

{
    "id": 67,
    "city": "Cincinnati",
    "state": "OH",
    "tweet": "https://twitter.com/P0kes/status/1267230163521191936",
    "file": "Cincinnati-EZYbHXOXkAEoFCa.jpeg",
    "youtube": "",
    "text": "police give notice that citizens are living in occupied territory, taking down the American flag outside the Justice Center and replacing it with their own"
  },

which is a jpeg vs a png, but this is also in there

{
    "id": 10,
    "city": "Minneapolis",
    "state": "MN",
    "tweet": "https://twitter.com/greg_doucette/status/1266352623608901634",
    "file": "Mpls-deOD6lzQ.jpeg",
    "youtube": "ftLzQefpBvM",
    "text": "Police arresting CNN reportes on air [h/t to @KSeyiA for link]"
  },

which looks like it should contain a jpeg, but in fact is a link to a twitter moment.