django-rest-framework icon indicating copy to clipboard operation
django-rest-framework copied to clipboard

Fix OpenAPI operation name plural appropriately

Open heavenshell opened this issue 4 years ago • 5 comments

Description

This PR is re-challenge of https://github.com/encode/django-rest-framework/pull/7447

I introduced inflection library to pluralize appropriately.

It would be very grateful to review this 😊

Thank you.

heavenshell avatar Jun 04 '21 15:06 heavenshell

Certainly going to be neater than what we're currently doing. I guess I don't have a problem with introducing this dependancy, given that it's pretty tiny. Anyone else?

lovelydinosaur avatar Jun 07 '21 09:06 lovelydinosaur

Thank you for reviewing!

I don't know why CI is failing... Could anyone help this? What should I do?

heavenshell avatar Jun 07 '21 13:06 heavenshell

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 28 '22 02:04 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 13 '22 22:07 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 21 '22 01:09 stale[bot]

the only build failure is a doc related failure, which does not seem to be related to this PR

auvipy avatar Nov 24 '22 07:11 auvipy

@auvipy Thank you so much! 👍

heavenshell avatar Nov 24 '22 16:11 heavenshell

pretty sure this broke the build by introducing a new hard dependency through the backdoor. see #8781

tfranzel avatar Nov 28 '22 00:11 tfranzel