LaRocheSurYonEventsBridge: initial version

[marius851000]

A site for a city where I live nearby.

I’m a bit unsure about 2 point:

1. The dates. There is no "post-time", and the start time of some event is 1970, certainly a placeholder. Thus, post time may not represent the way it is sorted (it isn’t explicit which one is used, but it seems to be roughly chronological)
2. Should I parse all pages, or only the first one? As of today, thay would make 5 pages to parse (including the first one). That seems reasonable, but I’m still a bit unsure that’s necessary. Will think a bit more about that, likely doing another PR if I actually implement that.

(also, thanks for this software. It was incredibly easy to write this despite my lack of experience with PHP)

[github-actions]

Pull request artifacts

Bridge	Context	Status
LaRocheSurYonEvents	1 untitled (pr)	❌ HTTP status 504 Gateway Time-out

last change: Monday 2023-10-16 06:13:42

[dvikan]

since these are events you need to decide how you want to structure the feed.

one option is to include in the feed only those events which are happening the next 7 days.

[ORelio]

1. I'd suggest to provide event date as text inside $item['content'], not as item['timestamp'] because item timestamp is supposed to indicate when the entry is published on the website, not the event date itself. Since it's not stated on the site, better leave it empty.

2. You already have 12 entries on first page, which looks more than enough to me. The software that reads your feed (aggregator/reader) will automatically keep old entries for you so you don't need to include all the pages in the feed.

[dvikan]

agreed with @ORelio

should probably not populate the timestamp field at all here. Unless you can find the actual publication date of event announcement.

[dvikan]

let know if u still here

[marius851000]

Sorry, I tend to loose focus too quickly. I'll ptobably return on it sooner or later. 16 oct. 2023 03:00:58 Dag ***@***.***>:

…

let know if u still here — Reply to this email directly, view it on GitHub[#3704 (comment)], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AFMKJZBUJ2YJX6WYS4WIW73X7SBMPAVCNFSM6AAAAAA5FDZTYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGU3TCOBVGY]. You are receiving this because you authored the thread. [Image de pistage][https://github.com/notifications/beacon/AFMKJZCLSSQ3GF5ZUIXKM5LX7SBMPA5CNFSM6AAAAAA5FDZTYKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTJDX4JA.gif]

[dvikan]

lemme know when u find time