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

Fixes date/location parsing from Papercall #39

Merged
merged 2 commits into from
Jun 19, 2019

Conversation

karllhughes
Copy link
Contributor

Hey @heiglandreas, thanks for the great work on this repo!

As mentioned here: joindin/callingallpapers-api#52

I noticed the event dates and locations were not being parsed properly. I'm guessing Papercall updated the DOM and broke this.

The fix was rather simple once I got the repo working locally. I added a couple notes to the Readme to help others who might work on on this project as well.

Please let me know if there's anything else I can do!

@heiglandreas
Copy link
Member

Thanks! Awesome work in finding and fixing the issue. I need to fix the CI issue later so worst case scenario would be that the parser is fixed tomorrow.

@heiglandreas heiglandreas merged commit ceb6fa4 into joindin:master Jun 19, 2019
@heiglandreas
Copy link
Member

Thanks! The new CLI is being deployed currently and the next run (around 01:00 UTC IIRC) will use your changes. So tomorrow the results should be fixed for papercall.io

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

Successfully merging this pull request may close these issues.

2 participants