Skip to content

Conversation

@bidi47
Copy link
Member

@bidi47 bidi47 commented Dec 2, 2025

No description provided.

Signed-off-by: bidi <bidi@apidemia.com>
Copy link
Member

@alexmerlin alexmerlin left a comment

Choose a reason for hiding this comment

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

Also, middlewares should be replaced with middleware across all pages.

Signed-off-by: bidi <bidi@apidemia.com>
@alexmerlin
Copy link
Member

alexmerlin commented Dec 2, 2025

Note

This is just a suggestion:

IF we are to modify the routes, we should definitely modify more than just fixing a typo.

We should modify all three routes like this:

  • flowMiddlewares -> flow-middleware
  • flowLibraries -> flow-libraries
  • flowLibrariesEmail -> flow-libraries-email

because camelCase is not too SEO friendly (and does not look too good either).

@arhimede can confirm if it's ok to modify those URLs

@arhimede
Copy link
Member

arhimede commented Dec 2, 2025

Note

This is just a suggestion:

IF we are to modify the routes, we should definitely modify more than just fixing a typo.

We should modify all three routes like this:

* `flowMiddlewares` -> `flow-middleware`

* `flowLibraries` -> `flow-libraries`

* `flowLibrariesEmail` -> `flow-libraries-email`

because camelCase is not too SEO friendly (and does not look too good either).

@arhimede can confirm if it's ok to modify those URLs

@bidi47 yes, let's modify the routes , to follow the standard

Signed-off-by: bidi <bidi@apidemia.com>
@bidi47
Copy link
Member Author

bidi47 commented Dec 3, 2025

i updated the route slugs
we also need to modify them manually on dotkernel.org's local.php after this PR goes live

Signed-off-by: bidi <bidi@apidemia.com>
@arhimede arhimede merged commit ae08845 into main Dec 3, 2025
18 checks passed
@alexmerlin alexmerlin deleted the flows branch December 3, 2025 11:00
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.

4 participants