Skip to content

fix: link Events section buttons to /event page - #267

Merged
Josephat-S merged 2 commits into
Open-Source-Kigali:developmentfrom
abhiramvsmg:fix/events-button-links
Jul 28, 2026
Merged

Josephat-S merged 2 commits into
Open-Source-Kigali:developmentfrom
abhiramvsmg:fix/events-button-links

Conversation

@abhiramvsmg

Copy link
Copy Markdown
Contributor

Fixed both PrimaryButton and SecondaryButton in the featured event card, changing to="" to to="/event". Previously, clicking either button routed back to the homepage instead of the events listing page. Now both "Register Now" and "Learn More" correctly navigate to /event.

Fixes #257

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

@abhiramvsmg is attempting to deploy a commit to the debfevlo's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ninzizah

Copy link
Copy Markdown
Collaborator

Hi @abhiramvsmg! Thanks for submitting this fix. The code changes are perfect!
However, we do all our active development and testing on the development branch, not main. Could you please change the base branch of this PR from main to development?
(You can do this by clicking the Edit button next to the PR title at the top, and changing the dropdown from main to development). Once you change it, we will merge this right away!

@abhiramvsmg
abhiramvsmg changed the base branch from main to development July 16, 2026 05:59
@abhiramvsmg

Copy link
Copy Markdown
Contributor Author

Done — switched the base branch to development. Thanks for the quick review!

@ninzizah

Copy link
Copy Markdown
Collaborator

Hi @abhiramvsmg! Thanks for submitting this fix. The code change is perfect and resolves the issue cleanly.

However, we do all active development and testing on the development branch instead of main. Could you please change the base branch of this PR from main to development?

You can do this by clicking the Edit button next to the PR title at the top, and changing the base dropdown from main to development

Once you change it, we will merge this right away! Thank you!

@Josephat-S Josephat-S left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great work

@Josephat-S
Josephat-S merged commit 6db54cd into Open-Source-Kigali:development Jul 28, 2026
1 check failed
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.

fix: Events section 'Register Now' and 'Learn More' buttons link to empty paths

3 participants