search

public-people / scrape-news

Scrape South African news
MIT License
12 stars 13 forks source link

Dailyvoice #3

Closed goose-life closed 6 years ago

goose-life commented 6 years ago

this one isn't quite working yet, but there's no error message; it just does the initial things and then 'finishes'. I assume the problem is in getting it to start properly?

screenshot from 2018-05-31 14-55-30

goose-life commented 6 years ago

I'm going to make some changes to the spider based on changes made to the timeslive spider, and will then push my changes so that the conflict should be resolved, fingers crossed.

jbothma commented 6 years ago

Sorry, forgot to explain the changes. Will do asap

On Sat, 02 Jun 2018, 21:21 edith, notifications@github.com wrote:

I'm going to make some changes to the spider based on changes made to the timeslive spider, and will then push my changes so that the conflict should be resolved, fingers crossed.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/public-people/scrape-news/pull/3#issuecomment-394110614, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOZGaHuSjE2qmTchCq61QHtmkkXrRHTks5t4uW4gaJpZM4UVJAZ .

goose-life commented 6 years ago

I went with using the text for the date in the end because the date 'content' was in UTC, so 'localizing' it made it wrong. I could have done something else as well I'm sure, let me know if I should!