Skip to content

Conversation

LomaxOnTheRun
Copy link
Contributor

@LomaxOnTheRun LomaxOnTheRun commented May 19, 2023

Reference to #31949.

This PR makes the no_append_slash decorator able to handle both sync and async views.

@LomaxOnTheRun LomaxOnTheRun changed the title Refs #31949 -- Made @no_append_slash work with async functions. Refs #31949 -- Made @no_append_slash work with async functions. May 19, 2023
@LomaxOnTheRun LomaxOnTheRun changed the title Refs #31949 -- Made @no_append_slash work with async functions. Refs #31949 -- Made @no_append_slash work with async functions May 19, 2023
@LomaxOnTheRun LomaxOnTheRun force-pushed the async-common-view-decorator branch from 3c30b1d to 5c816fc Compare May 19, 2023 15:20
@LomaxOnTheRun LomaxOnTheRun marked this pull request as ready for review May 19, 2023 15:54
@LomaxOnTheRun LomaxOnTheRun force-pushed the async-common-view-decorator branch from 5c816fc to 9bffeed Compare May 22, 2023 02:11
@felixxm felixxm changed the title Refs #31949 -- Made @no_append_slash work with async functions Refs #31949 -- Made @no_append_slash decorator to work with async functions. May 23, 2023
@felixxm felixxm force-pushed the async-common-view-decorator branch from 9bffeed to 4a2302f Compare May 23, 2023 07:59
Copy link
Member

@felixxm felixxm left a comment

Choose a reason for hiding this comment

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

@LomaxOnTheRun Thanks 👍

@felixxm felixxm force-pushed the async-common-view-decorator branch from 4a2302f to 23abec9 Compare May 23, 2023 08:04
@felixxm felixxm merged commit 23abec9 into django:main May 23, 2023
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.

2 participants