Muhammet Ali Topcu's Individual Contribution Report 2 - bounswe/bounswe2023group1 GitHub Wiki

Member: Muhammet Ali Topcu

Group: Group 1 - Disaster Response Platform

Responsibilities:

  • Notification Entity and its Service Implementation and Database Table Creation.
  • Enabling HTTPS with SSL/TLS certificate
  • Updating Dockerfile and Creating docker-compose.yml
  • Authorizing victims to be able to view resources
  • Backend Null Check and Auth Errors Fix
  • Backend CORS Error Fix
  • AuthService Unit Test Fix
  • Firebase Cloud Messaging Backend Implementation (Postponed)
  • Handling the Notification Cases (Postponed)
  • Set Up Jenkins for CI/CD Integration for Backend (Postponed)

Main contributions:

  • Notification database relation is created in the database with relevant relations. In the code, there should be some points to send notification to the user. Some of them are covered, and in the future, there will be some other parts where the application will need to send notification, too.
    Issues: #396

  • Enabled HTTPS with SSL/TLS certificate so that our application will provide security.
    Issues: #430

  • Updated the Dockerfile. Created two separate Dockerfiles, one for local and one for remote deployment. In addition, created docker-compose.yml to run the application easily with one or two commands in the local.
    Issues: #431

  • Victims should be able to view resources on the map. So, for that endpoint, added that role to the authorized access list.
    Issues: #476

  • Backend Null Check and Auth Errors. In this issue, convertion from entity to dto was causing NullPointerException, fixed. In addition, since /error page is not authorized, the application was giving unauthorized access error. It is fixed, too.
    Issues: #478

  • Bugfix Backend CORS Error. When frontend team wanted to send request from their local to our backend application that is deployed, they received CORS error. I fixed the issue with adding default CORS configuration.
    Issues: #484

  • AuthService Unit tests were already written. After some part of the code changed, fixed the relevant points in the tests.
    Issues: #490

  • Firebase Cloud Messaging Backend Implementation API Integration would be done together with the Notification Service Implementation. However, since the mobile team postponed it and I need a token from them, I also postponed this implementation.
    Issues: #401

  • Handling the Notification Cases. Postponed because of the same reason stated above.
    Issues: #427

  • Set Up Jenkins for CI/CD Integration for Backend. Spent around 15 hours to succeed, but couldn't. So, we may continue to deploy our app manually with a bash script written by me. (Postponed)
    Issues: #446

Management related significant issues:

  • Enabling HTTPS with SSL/TLS certificate.
  • Manual deployment of the backend after merged PRs.
  • Created API Documentation and provided the link to the API in the milestone report.
    Issues: #521
  • Contributed to writing our progress for our requirements in the milestone report.
    Issues: #520
  • Generated unit test report for backend and added in the milestone report. In addition, contributed to the test plan writing in docs.
    Issues: #522, #523

Pull Requests:

  • PRs created by me:

    • Pull request for Notification Service and Database Implementation.
      PR: #400
      Issue: #396

    • Pull request for updating dockerfile and creating docker composeyml.
      PR: #441
      Issue: #431

    • Pull request for giving victims authorization to view resources.
      PR: #477
      Issue: #476

    • Pull request for Backend Null Check and Auth Errors Fix.
      PR: #479
      Issue: #478

    • Pull request for CORS Configuration.
      PR: #489
      Issue: #484

    • Pull request for unit test fix for AuthService class.
      PR: #494
      Issue: #490

  • PRs reviewed and merged by me:

    • Merged pull request for Finalizing Request.
      PR: #387
      Issue: #377

    • Merged pull request for Resource API Enhancement.
      PR: #393
      Issue: #376

    • Merged pull request for Get update profile.
      PR: #448
      Issue: #447

    • Merged pull request for Entities to dto.
      PR: #455
      Issue: #454

    • Merged pull request for Add creation date to dtos.
      PR: #471
      Issue: #470

    • Merged pull request for Location Improvement.
      PR: #492
      Issue: #392

Additional information:

  • Above, Enabling HTTPS with SSL/TLS certificate, updating Dockerfile and creating docker-compose.yml, Set Up Jenkins for CI/CD Integration for Backend issues were not planned. 2 of them implemented successfully, but the last one couldn't. We may continue to deploy our app manually.