Skip to content

feat: allow to read event_ids from hash url to render a custom schedule - #209

Merged
smarcet merged 9 commits into
mainfrom
feature/schedule-events-id
Aug 20, 2026
Merged

feat: allow to read event_ids from hash url to render a custom schedule#209
smarcet merged 9 commits into
mainfrom
feature/schedule-events-id

Conversation

@tomrndom

@tomrndom tomrndom commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

ref: https://app.clickup.com/t/9014802374/86bbcamxh

What kind of change does this PR introduce?

  • allow to read event_ids from hash url to render a custom schedule

Does this PR introduce a breaking change?

No

What needs to be documented once your changes are merged?

Nothing

Test Notes

https://deploy-preview-209--qa-fnvirtual.netlify.app/a/schedule#event_ids=6689,6715,6782

… schedule

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@netlify

netlify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Deploy Preview for qa-fnvirtual ready!

Name Link
🔨 Latest commit 8ee1e0f
🔍 Latest deploy log https://app.netlify.com/projects/qa-fnvirtual/deploys/6a86156ba2a76d0007c8c756
😎 Deploy Preview https://deploy-preview-209--qa-fnvirtual.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@smarcet
smarcet requested review from gcutrini, santipalenque and smarcet and a lite review from Copilot and removed request for santipalenque August 18, 2026 16:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread src/reducers/schedule-reducer.js
Comment thread src/actions/schedule-actions.js Outdated
Comment thread src/actions/schedule-actions.js

@smarcet smarcet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tomrndom please review

…S action

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@tomrndom
tomrndom requested a review from smarcet August 18, 2026 19:27
Comment thread src/actions/schedule-actions.js Outdated
Comment thread src/templates/schedule-page.js

@smarcet smarcet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tomrndom please review

…ecode event_ids, add fallback value

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
…d decoded event_ids

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>

@smarcet smarcet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smarcet
smarcet merged commit ee3a8d0 into main Aug 20, 2026
6 checks passed
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.

4 participants