Add test case to show incorrect CoreAPI schema generation and fix the issue (#6007) #6685
Conversation
yagggi
changed the title
| description=desc_1 | ||
| ) | ||
| description=desc_0), | ||
| 'detail': coreapi.Link( |
There was a problem hiding this comment.
I'm not entirely sure I agree with this being the right location for this link. This seems like it should represent /from-routercollision/detail/detail/, but instead it is representing one layer up.
Additionally, this breaks backwards compatibility with how we handle naming collisions. (See #5486 for where this was first implemented)
| url='', | ||
| title='Single Prefix', | ||
| content={ | ||
| 'route1': { |
There was a problem hiding this comment.
I'm probably reading this wrong somewhere in the code, but shouldn't this be prefix? Since these are both custom routes under the same prefix path, shouldn't they be common links in the same dictionary under prefix?
carltongibson
changed the title
|
Closing this as stale. CoreAPI schema generation is not longer maintained. It's still available for people who are using it but will be removed in a future version. |
Description
Fix #6007