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

Fix parsing of google calendar data. #72

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nmb
Copy link
Contributor

@nmb nmb commented Sep 4, 2019

No description provided.

Mikael Borg added 2 commits September 4, 2019 16:01
Change "datetime" to "dateTime" in order to parse events from google
calendar properly.
@varemo
Copy link

varemo commented Sep 19, 2019

Hej, will anyone have time to fix this? Can I help in any way? I have a course running in November and it is displayed with incorrect information on TeSS (https://tess.elixir-europe.org/events/nbis-course-prel-reproducible-research) so it would be great if it is possible to fix soon.

Cheers

/Leif

@njall
Copy link
Contributor

njall commented Sep 19, 2019

Sorry! I'm juggling too many balls at the moment and have let this slip.

I've run the pull request.

Sadly, it does not fix your event. Attached is a screenshot. A green tick means the event is being scraped today. A red exclamation mark means it's stale/inaccessible. This means the scraper is not able to find/update your event using this PR version of the scraper. I'm on leave from now until Wednesday next week. Maybe I could look into it when I get back? Alternatively you could try improving the script and try testing it using our production test server. You will need to sign up for an account on here https://tess.oerc.ox.ac.uk/ and give me your username so I can give you API write access

For now, I could delete this event (and other stale ones) as it looks like your event is being pulled in correctly from another source; SciLifeLab.

Screenshot 2019-09-19 at 16 21 02

@varemo
Copy link

varemo commented Sep 20, 2019

Thanks @njall for your quick reply! The last link you include indeed seems to pull in the event correctly. So I guess it makes sense to delete the original incorrect one. Regarding fixing the script I am not sure I have the background to work on it, but let me know if I can help in any other way. Thanks again!

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.

3 participants