Skip to content

Conversation

@tbeadle
Copy link

@tbeadle tbeadle commented Aug 21, 2024

No description provided.

@tfranzel
Copy link
Owner

Hi, thanks for the PR. I have not looked at this in a while. How much support do we gain with this change? Are there other changes necessary?

@tbeadle
Copy link
Author

tbeadle commented Aug 27, 2024

I read through #947 and I see that there is a lot of history going on with the decision whether or not to support adrf. I'm sure this change is not everything that is needed to completely support it. It just fixed the one issue that I was having.

@bartvanandel
Copy link
Contributor

Nice work! I want your proposed changes merged as well, we're in the same situation. I tried to use CodeSpaces to review your changes and make some improvements, well, see my PR for your PR here: tbeadle#1

@tfranzel would be nice if you could integrate those changes (including my proposed improvements) in the next version. It may not yet cover all situations for adrf, but ...

How much support do we gain with this change?

... works for us!

Improvements to PR for getting drf-spectacular to work with adrf.
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.

3 participants