digital-preservation / PRONOM_Research

28 stars 10 forks source link

Add ICS schedules for UK and AET meetings #23

Closed ross-spencer closed 1 year ago

ross-spencer commented 1 year ago

A proposal to add an ICS calendar to the repository for AET and GMT times based on the drop-in.md file. The text in the drop-in MD file has been updated so that it is easier to maintain and only links to the calendar file.

NB. .ics is plain-text and we might want to review it further to see if there are any customization anyone thinks would be a good idea. The calendar was created using: http://ical.marudot.com/

NB. UK and AET times have been transposed to CET times but should be the same, but it would be helpful if folks can confirm.

The modified README.md can be better reviewed here.

ross-spencer commented 1 year ago

:thinking: it might not be such a good idea, it doesn't seem that easy to make the ICS file clickable. Will have a think about this some more.

richardlehane commented 1 year ago

tested and works OK if you click on the download button

image

Suggest adding the URL to the description field as well (in Outlook it just said the location was "Online" without any link to the Teams call)

On Thu, May 25, 2023 at 3:53 PM Ross Spencer @.***> wrote:

🤔 it might not be such a good idea, it doesn't seem that easy to make the ICS file clickable. Will have a think about this some more.

— Reply to this email directly, view it on GitHub https://github.com/digital-preservation/PRONOM_Research/pull/23#issuecomment-1562951722, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABGN3W32A2WEYRFIMR7HQDXH5P7LANCNFSM6AAAAAAYO3MOVI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ross-spencer commented 1 year ago

Suggest adding the URL to the description field as well (in Outlook it just said the location was "Online" without any link to the Teams call)

Totally - will make it so. Definitely some difference between calendar approaches here.

ross-spencer commented 1 year ago

cc @AndreaHri @tnafrancesca @andreykotov91 following on from yesterday, this commit adds a calendar entry that folks may use for the two times. The calendar will simply repeat until a user opts to delete it. Perhaps some text around that may be helpful? Otherwise, in the calendar entry folks will also find a simple description and as Richard suggested, the URL to the teams meeting.

tnafrancesca commented 1 year ago

This is great- thank you so much @ross-spencer . Changes made are clickable links to add the PRONOM Open Drop-In dates to personal calendars

prettybits commented 1 year ago

@ross-spencer I just had a closer look at the URL in the iCalendar files as I had trouble yesterday joining the meeting by following the link in the calendar event. It looks like the URL has been double percent-encoded, so my browser after decoding still tries an invalid URL and is redirected by Teams to an error page.

Current value: https://teams.microsoft.com/l/meetup-join/19%253ameeting_OThkMzIwMWItYTUzYS00NDIxLTgzNTEtMzAwMGQzYWZhNmJi%2540thread.v2/0?context=%257b%2522Tid%2522%253a%2522f99512c1-fd9f-4475-9896-9a0b3cdc50ec%2522%252c%2522Oid%2522%253a%2522e02319b9-2b3a-4408-86ee-7cd0ba62ed9f%2522%257d

After decoding: https://teams.microsoft.com/l/meetup-join/19%3ameeting_OThkMzIwMWItYTUzYS00NDIxLTgzNTEtMzAwMGQzYWZhNmJi%40thread.v2/0?context=%7b%22Tid%22%3a%22f99512c1-fd9f-4475-9896-9a0b3cdc50ec%22%2c%22Oid%22%3a%22e02319b9-2b3a-4408-86ee-7cd0ba62ed9f%22%7d

Decoding a second time gives https://teams.microsoft.com/l/meetup-join/19:meeting_OThkMzIwMWItYTUzYS00NDIxLTgzNTEtMzAwMGQzYWZhNmJi@thread.v2/0?context={"Tid":"f99512c1-fd9f-4475-9896-9a0b3cdc50ec","Oid":"e02319b9-2b3a-4408-86ee-7cd0ba62ed9f"} which is the link also used in the document which ultimately worked.

I think this happened because the generator you used unconditionally encodes the URL you put there, but when copying a link target from the browser it already gives you an encoded URL, leading to the double encoding issue. I can open a PR for this if you don't see an issue with my analysis?

ross-spencer commented 1 year ago

@prettybits that sounds messy. if you can clean it up then please go for it!