Skip to content

fix: case-sensitivity in apps path#6552

Merged
emrysal merged 2 commits intomainfrom
6548-cal-820-lowercase-apps-calcomappssignal-404-whereas-signal-works
Jan 21, 2023
Merged

fix: case-sensitivity in apps path#6552
emrysal merged 2 commits intomainfrom
6548-cal-820-lowercase-apps-calcomappssignal-404-whereas-signal-works

Conversation

@G3root
Copy link
Copy Markdown
Contributor

@G3root G3root commented Jan 18, 2023

What does this PR do?

Fixes #6548

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

  • Test A
  • Test B

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't performed a self-review of my own code and corrected any misspellings
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my PR needs changes to the documentation
  • I haven't checked if my changes generate no new warnings
  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

Loading
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.

[CAL-820] lowercase apps: cal.com/apps/Signal 404 (whereas /signal works)

4 participants