Skip to content
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.

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

Support upper case letters in route definition #3210

Closed
marsteg opened this issue Apr 9, 2024 · 1 comment
Closed

Support upper case letters in route definition #3210

marsteg opened this issue Apr 9, 2024 · 1 comment
Labels

Comments

@marsteg
Copy link
Contributor

marsteg commented Apr 9, 2024

Background

Motivation
The CF-Acceptance-Tests (CATS) are creating apps, containing capital letters e.g. CATS-2-APP-eae0787d0d5dd6ba. For compatibility reasons we should support it even if the spec states otherwise.

Goals
Allow apps with routes with capital letters to be pushed successfully without modifying the manifest.

Action to take

No response

Impact

Improve CF compatibility.

Dev Notes

No response

@marsteg marsteg added the chore label Apr 9, 2024
@github-project-automation github-project-automation bot moved this to 🧊 Icebox in Korifi - Backlog Apr 9, 2024
@georgethebeatle georgethebeatle moved this from 🧊 Icebox to ⏳ Review in progress in Korifi - Backlog Apr 11, 2024
@georgethebeatle georgethebeatle moved this from ⏳ Review in progress to ⌛ Reviewer approved in Korifi - Backlog Apr 11, 2024
@marsteg
Copy link
Contributor Author

marsteg commented Apr 11, 2024

solved in: #3211

@marsteg marsteg closed this as completed Apr 11, 2024
@github-project-automation github-project-automation bot moved this from ⌛ Reviewer approved to ✅ Done in Korifi - Backlog Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

1 participant