[change] Updated download batch CSV URL #373

* URL generated with radius:serve_private_file was not consistent with the rest of the API URLs.
* Changed Download batch CSV API endpoint from radiusbatch/csv/<uuid>/filename.csv to radius/organization/<slug:slug>/batch/<uuid:pk>/csv/filename.csv and added the same in API documentation as well.

Related to #373

Co-authored-by: Dishant Sethi <dishantsethi@Dishants-MacBook-Air.local>
Co-authored-by: Federico Capoano <f.capoano@openwisp.io>

docs/source/user/api.rst

@@ -766,3 +766,22 @@ When using this strategy, in the response you can find the field
 (example: ``[['username', 'password'], ['sample_user', 'BBuOb5sN']]``)
 and the field ``pdf_link`` which can be used to download a PDF file
 containing the user credentials.
+
+Batch CSV Download
+-------------------
+
+.. code-block:: text
+
+ /api/v1/radius/organization/<organization-slug>/batch/<id>/csv/<filename>
+
+Responds only to **GET**.
+
+Parameters:
+
+=============== ===============================
+Param Description
+=============== ===============================
+slug string
+id string
+filename string
+=============== ===============================

openwisp_radius/base/models.py

@@ -830,7 +830,13 @@ def __str__(self):
 
 
 def _get_csv_file_location(instance, filename):
- return os.path.join(str(instance.organization.pk), filename)
+ return os.path.join(
+ str(instance.organization.slug),
+ 'batch',
+ str(instance.organization.pk),
+ 'csv',
+ filename,
+ )
 
 
 class AbstractRadiusBatch(OrgMixin, TimeStampedEditableModel):

openwisp_radius/settings.py

@@ -135,7 +135,7 @@ def get_settings_value(option, default):
  )
 
 # Path of urls that need to be refered in migrations files.
-CSV_URL_PATH = 'api/v1/radiusbatch/csv/'
+CSV_URL_PATH = 'api/v1/radius/organization/'
 CALLED_STATION_IDS = get_settings_value('CALLED_STATION_IDS', {})
 CONVERT_CALLED_STATION_ON_CREATE = get_settings_value(
  'CONVERT_CALLED_STATION_ON_CREATE', False