Weekly Progress Report - OpenWISP REST API [GSoC'21]

169 views
Skip to first unread message

Manish Shah

unread,
May 21, 2021, 2:46:37 PM5/21/21
to OpenWISP Google Summer of Code
May 17, 2021 - May 21, 2021:-

- List of tasks I have worked on:-
    - Added tests for the PKI App API endpoints[openwisp-controller] PR #455
    - Updated the REST API sections with the endpoints of the PKI App
      in the openwisp-controller README. PR #455

- Tasks I will be working on next week:-
  - Resolve API: URL name space and view names #467 [openwisp-controller]
  - Will try to open a PR in Django Rest Framework Repo to implement the view

- Doubts/Questions:-
  - Will have to discuss and make the issues clearer which are presently being faced with the Config App API endpoints.

- Blocker:
 
- None

Kind Regards,
Manish Kumar Shah
Message has been deleted

Manish Shah

unread,
May 29, 2021, 12:25:31 AM5/29/21
to OpenWISP Google Summer of Code
May 24, 2021 - May 28, 2021:-

- List of tasks I have worked on:-
  - Updated the Kanban Board with the issues, explaining about the endpoints I have thought of introducing in various modules. Project Board
  - Resolved the View Permission issue. PR #251
  - Changed the existing API endpoints names and namespaces for `Config` App. Pr #473
  - Opened a similar PR to PR #251 in DRF repo to resolve the permission issue with `DjangoModelPermissions` Class PR #8009
  - Renamed the API endpoints name and namespaces for `PKI` App. Commit 7de743b
  - Went through the Codebase of the `GEO` App and studied a bit about the existing API endpoints.

- Tasks I will be working on next week:-
  - Will open a PR for API of the `GEO` App.
  - Will resolve if any change requests in the opened PR's

- Doubts/Questions:-
  - Asking on the Gitter Channel

- Blocker:
  - None

- PR's Ready for review:-


Sorry for being late.

Manish Shah

unread,
Jun 4, 2021, 1:49:25 PM6/4/21
to OpenWISP Google Summer of Code
May 31, 2021 - June 4, 2021:-

List of tasks I have worked on:-
  - Updated the docs and simplified code in encode's DRF repo view-permission PR #8009
  - Tried moving the download CRL View function to a separate Mixin class PR #117 to make it reusable for the the REST API endpoint of PKI APP
  - Hence Update REST API PKI with the new mixin class PR #455
  - Renamed the GEO APP API endpoint names and namespace PR #474
  - Updated documentation, Added Tests, and simplified Code in the for extending the DjangoModelPermissions class PR #251
  - Implemented the view permission with the extended  DjangoModelPermissions class to the Config App API endpoints. PR #476
  - Introduced class based view for fetching the CRL of CA with the PKI App API endpoint Commit 3b7674b
  - Improved the GEO App API Endpoints pattern PR #483


Tasks I will be working on next week:-
  - Will try to open a PR for the API of the GEO App.

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Federico Capoano

unread,
Jun 6, 2021, 7:57:57 PM6/6/21
to Manish Shah, OpenWISP Google Summer of Code
Thanks Manish, it's going well in your project too.


Please follow these suggestions (valid for anyone else reading this list):

- whenever you change code, ensure you update docs as well (see Ajay's latest review)
- whenever you add new features, changes, bugfixes, write tests that fail without your fixes/changes, this gives us the highest probability tests will catch inadvertent mistakes (from yourself, me or anybody else) which may break the functionality you are adding/fixing inadvertently

Please these suggestions above must become automatic without the need of us reminding every student.

Best regards
Fed

--
You received this message because you are subscribed to the Google Groups "OpenWISP Google Summer of Code" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openwisp-gso...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openwisp-gsoc/6fece336-3c23-436e-b779-cca2d1812011n%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Manish_18 Shah_3019

unread,
Jun 11, 2021, 3:44:03 PM6/11/21
to OpenWISP Google Summer of Code
June 7, 2021 - June 11, 2021:-

List of tasks I have worked on:-
  - REST API for GEO App. [Feature Branch]
  - Improve API mixin for organization look up field in parent model PR #258
  - Fixed minor chnage in Issue #142
  - Chnaged API endpoints of GEO App PR #483
  - Resolved requested chnages in PKI APP REST API PR #455

Tasks I will be working on next week:-
  - Will try to open a PR for the API of the GEO App. [Progress]
  - Will resolve issue #142 [ openwisp-firmware-upgrader]
  - will try to resolve any changes if requested in PR #455
  - will make chnages if any requested in PR #258

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Manish_18 Shah_3019

unread,
Jun 18, 2021, 3:11:58 PM6/18/21
to OpenWISP Google Summer of Code
June 14, 2021 - June 18, 2021:-

List of tasks I have worked on:-
  - REST API for the connection app Working Branch
  - REST API for GEO App PR #494
  - Resolved the KeyError in FilterSerializerByOrgManaged mixin class PR #260
  - Improved API mixins for organization field lookup in parent model PR #258 Co-Authored By: Federico Capoano


Tasks I will be working on next week:-
  - Open a PR for REST API of connection APP. Issue #464
  - Will try to resolve the Issue #256
  - Implement mixin classes from openwisp-users in openwisp-firmware-upgrader module. Issue #142
  - Resolve if any chnage requests in the the existing open PR's.

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Manish Shah

unread,
Jun 25, 2021, 3:26:08 PM6/25/21
to OpenWISP Google Summer of Code
June 21, 2021 - June 25, 2021:-

List of tasks I have worked on:-
  - Added the missing functionality for importing existing Ca/Cert in the PKI App API endpoints PR #455
  - Reported a Issue for redundant code in network-topology Module Issue #111
  - Worked and tried debugging the assertNumQueries for the reasons behind increasing number of queries Issue #256
  - Worked on Firmware-Upgrader for improvements in the existing branch Feature Branch 


Tasks I will be working on next week:-
  - Will try to Open A PR for the API endpoint of openwisp-network-topology module.
  - Will resolve if any changes requested in the Open PR's
  - Will try to add the import existing cert/ca feature to the list/create endpoints.
  - Will try to resolve the half done work in Firmware-Upgrader Module for the improvements of the API endpoints.

Doubts/Questions:-
  - Can I first work on the openwisp-network-topology module, and then move to openwisp-users module?

Blocker:
  - None

PR's ready for review:-

Ajay Tripathi

unread,
Jun 30, 2021, 12:53:01 PM6/30/21
to OpenWISP Google Summer of Code
Hi,

Doubts/Questions:-
  - Can I first work on the openwisp-network-topology module, and then move to openwisp-users module?

Users module will be difficult and would require time, pushing the difficult work for the end can be a risk, so
I would not recommend the same.
What do you think about the risk?


Best,
Ajay Tripathi

Manish Shah

unread,
Jul 2, 2021, 3:02:23 PM7/2/21
to OpenWISP Google Summer of Code
June 28, 2021 - July 2, 2021:-

List of tasks I have worked on:-
  - Moved the import existing CA/Certs functionality to their respective list API endpoints in the PKI App. PR #455
  - Resolved the requested changes in the `KeyError` Issue PR in the openwisp-user module. PR #260
  - Removed the redundant code for checking permissions for GET request in the `openwisp-network-topology` module. PR #112
  - Worked a bit on the API endpoints for the `openwisp-network-topology` module, but halted this work and started the work for the endpoints of `openwisp-users` module. (Didn't push the code)
  - Started the work for the API endpoints for the `openwisp-users` module, introduced the list & post endpoints for the organizations model. PR #264
  - Added tests for the above endpoints and introduced the endpoints for `User` list. Commit 34b2855

Tasks I will be working on next week:-
  - Will try to complete at least 85% of the work in the `openwisp-users` Module. PR #264
  - Will work on chnage requests if any in the present Open Pr's.

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-
  - Kanban Board (Need's Review Column)

Manish Shah

unread,
Jul 9, 2021, 3:11:55 PM7/9/21
to OpenWISP Google Summer of Code
July 5, 2021 - July 9, 2021:-

List of tasks I have worked on:-
  - Worked on `DeviceConnection` model for the API Endpoint of `Connection` App. PR #497
  - Worked on GEO app for the API Endpoint. PR #494
  - Resolved the requested changes in the `PKI` App. PR #455
  - Worked on the `Users` module to add a logic to filter user's for the API endpoint. PR #264

Tasks I will be working on next week:-
  - Will try to finish up the work of GEO app API endpoints. PR #494
  - Will start with the work on User's module. PR #264
  - If possible will parallely work on `network-topology` module Issue #109

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Manish Shah

unread,
Jul 16, 2021, 2:38:22 PM7/16/21
to OpenWISP Google Summer of Code
July 12, 2021 - July 16, 2021:-

List of tasks I have worked on:-
  - Introduced the CRUD API endpoints for the models of `openwisp-network-topology` module and updated the docs. PR #113
  - Worked on the `DeviceLocation` endpoint in the `GEO` app. commit
  - Added the `delete` option to the existing `devicelocation` endpoint in the `GEO` App. PR #494
  - Added the `User` and `organization` model endpoint for the `openwisp-users` module. PR #264
  - Fixed the requested chnages in the PKI APP API endpoint PR, for the extensions field returning `null` and CA ID not displayed in the Cert Detail endpoints. PR #455


Tasks I will be working on next week:-
  - Will work on the API endpoints for the `openwisp-users` module PR #264
  - Will parallelly try to work on the API endpoints, by adding tests and others for the `openwisp-network-topology` module PR #113
  - Will try to resolve any change requests if any in the existing PR's

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Manish Shah

unread,
Jul 23, 2021, 1:35:56 PM7/23/21
to OpenWISP Google Summer of Code
July 19, 2021 - July 23, 2021:-

List of tasks I have worked on:-
  - Added the required endpoints for `revoke` and `renew` of `CA` and `Cert`, added tests, Updated the docs for the `PKI` App. commit
  - Fixed minor bug in the organization list API endpoint. PR #264
  - Nested OrgnaizationUser, EmailAddress with the User Serialzier for list API endpoint. PR #264
  - Started the work for the CRUD API endpoint for the `Group` Model PR #264
  - Minor code improvements in the PKI sertializers. PR #455


Tasks I will be working on next week:-
  - Topology module, Users module

Doubts/Questions:-
  - Asking on the Gitter Channel

Blocker:
  - None

PR's ready for review:-

Manish Shah

unread,
Jul 30, 2021, 3:46:49 PM7/30/21
to OpenWISP Google Summer of Code
July 26, 2021 - July 30, 2021:-

List of tasks I have worked on:-
  - Introduced Email API endpoint for User's to make/unmake primary, verified, change email's. PR #264
  - Introduced the required API endpoint, of Group Model, added tests, updated docs. PR #264
  - Introduced API endpoint for changing passwords of users, added tests, updated docs PR #264
  - Resolved minor change requests in change password endpoint feature. PR #264
  - Added tests for the API endpoints for Node & Link Model in openwisp-network-topology module. PR #113
  - Resolved minor merge conflicts in GEO App API Pr. PR #494


Tasks I will be working on next week:-
  - Openwisp-user's module & Openwisp-network-topology module

Manish Shah

unread,
Aug 6, 2021, 3:04:34 PM8/6/21
to OpenWISP Google Summer of Code
August 2, 2021 - August 6, 2021:-

List of tasks I have worked on:-
  - Added all the required REST API endpoints in the `openwisp-users` module. PR #264
  - Resolved the bug in PKI App, API endpoints not included in main controller.urls file PR #514
  - Resolved requested changes in GEO App PR #494
  - Added the create operation feature for the topology list endpoint. PR #113


Tasks I will be working on next week:-
  - Network-Topology Detail View, and change requests if any from the existing
    ready for review PR's from the ready for review column.

Manish Shah

unread,
Aug 13, 2021, 2:31:21 PM8/13/21
to OpenWISP Google Summer of Code
August 9, 2021 - August 13, 2021:-

- List of tasks I have worked on:-
  - Fixed requested changes in the GEO App API PR #494
  - Improvements in the change password endpoint of User API endpoint PR #264
  - Opened PR to take into effect the present mixins and classes from openwisp-users module in firmware-upgrader module PR #150
  - Optimised the number of queries hit in openwisp-users API PR #264
  - Resolved a minor bug in filtering organization in the mixin class `FilterSerializerByOrgManaged` PR #269
  - Completed openwisp-network-topology API endpoints work PR #113

- Tasks I will be working on next week:-
  - Resolve change requests in existing Open PR's.
  - Work on the bugs of existing open PR's.
  - Take up new issue if possible.

Manish Shah

unread,
Aug 24, 2021, 10:05:24 AM8/24/21
to OpenWISP Google Summer of Code
Hello everyone,
Recently I have published my Google Summer of Code 2021 project report. You can find it here:- OpenWISP REST API: GSoC’21 Project Report
Please give it a read. 😊

Regards,
Manish
Reply all
Reply to author
Forward
0 new messages