-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[change!] Modified API endpoints #106 #107
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As commit prefix please use [change!]
, the exclamation mark stands for backward incompatible.
path('subnet/<str:pk>/', api_views.subnet, name='subnet'), | ||
path('subnet/<str:subnet_id>/hosts/', api_views.subnet_hosts, name='hosts'), | ||
path('ip-address/<str:pk>/', api_views.ip_address, name='ip_address'), | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in other modules, we usually just define the prefix in the root urlconf or something similar, so I suggest doing that to maintain consistency. Please add ipam
here:
openwisp-ipam/openwisp_ipam/urls.py
Line 14 in 7edd0c1
path('api/v1/', include((get_api_urls(api_views), 'ipam'), namespace='ipam')), |
Changed API endpoints from "/api/v1/*" to "/api/v1/ipam/*" Closes #106
7edd0c1
to
0f1504f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed API endpoints from "/api/v1/" to "/api/v1/ipam/"
Fixes #106