Skip to content
This repository was archived by the owner on Apr 2, 2025. It is now read-only.

extract route names to constants #146

Merged
merged 3 commits into from
Feb 25, 2025
Merged

extract route names to constants #146

merged 3 commits into from
Feb 25, 2025

Conversation

philvarner
Copy link
Contributor

Related Issue(s):

  • n/a

Proposed Changes:

  1. Extract route names to constants so they can be reused in link generation more easily

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@philvarner philvarner changed the title extract route names to contants extract route names to constants Feb 24, 2025
@philvarner philvarner marked this pull request as ready for review February 24, 2025 20:15
c-wygoda
c-wygoda reviewed Feb 25, 2025
View reviewed changes
@philvarner philvarner merged commit 45fd13b into main Feb 25, 2025
3 checks passed
@philvarner philvarner deleted the pv/extract-route-names-to-constants branch February 25, 2025 16:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pjhartzell pjhartzell Awaiting requested review from pjhartzell

@jkeifer jkeifer Awaiting requested review from jkeifer

@tylanderson tylanderson Awaiting requested review from tylanderson

1 more reviewer

@c-wygoda c-wygoda c-wygoda approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@philvarner @c-wygoda