这是indexloc提供的服务，不要输入任何密码
Skip to content

fix: allow 'index' keyword in routes name #647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 15, 2025
Merged

fix: allow 'index' keyword in routes name #647

merged 1 commit into from
Mar 15, 2025

Conversation

Valerioageno
Copy link
Member

Checklist

Related issue

Fixes #644

Overview

@Valerioageno Valerioageno requested a review from jacobhq March 15, 2025 09:45
@Valerioageno Valerioageno self-assigned this Mar 15, 2025
@github-actions github-actions bot added the rust Requires rust knowledge label Mar 15, 2025
@Valerioageno Valerioageno marked this pull request as ready for review March 15, 2025 09:52
jacobhq
jacobhq approved these changes Mar 15, 2025
View reviewed changes
Copy link
Contributor

@jacobhq jacobhq left a comment

Choose a reason for hiding this comment

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

Looks good!

@Valerioageno Valerioageno merged commit 2301083 into main Mar 15, 2025
29 checks passed
@Valerioageno Valerioageno deleted the allow-index-in-routes-name branch March 15, 2025 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacobhq jacobhq jacobhq approved these changes

Assignees

@Valerioageno Valerioageno

Labels
rust Requires rust knowledge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug]: API route containing index at the start of a path segment is not working
2 participants
@Valerioageno @jacobhq