Skip to content

Conversation

@Attigala
Copy link

PR checklist

TaskId: (https://github.com/SkappHQ/skapp/issues/[id])

Summary

How to test

Project Checklist

  • Changes build without any errors
  • Have written adequate test cases
  • Done developer testing in
    • Chrome
    • Firefox
    • Safari
  • Code is formatted with npm run format
  • Code is linted with npm run check-lint
  • No unnecessary comments left in code
  • Made corresponding changes to the documentation

Other

  • New atomic components added
  • New molecules added
  • New pages(routes) added
  • New dependencies installed

PR Checklist

  • Pull request is raised from the correct source branch
  • Pull request is raised to the correct destination branch
  • Pull request is raised with correct title
  • Pull request is self reviewed
  • Pull request is self assigned
  • Suitable pull request status labels are added (ready-for-code-review)

Additional Information

@CLAassistant
Copy link

CLAassistant commented Jul 29, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 5 committers have signed the CLA.

✅ Attigala
❌ devni-a
❌ brp-imalsha
❌ milindaRootcode
❌ Dlnmdu
You have signed the CLA already but the status is still pending? Let us recheck it.

brp-imalsha and others added 2 commits July 29, 2025 16:51
…le-to-view-the-team-objectives' into feat/643-as-a-user-I-must-be-able-to-view-the-team-objectives

# Conflicts:
#	backend/src/main/java/com/skapp/community/okrplanner/controller/v1/KeyResultTypeController.java
#	backend/src/main/java/com/skapp/community/okrplanner/controller/v1/TeamObjectivesController.java
#	backend/src/main/java/com/skapp/community/okrplanner/mapper/TeamObjectiveMapper.java
#	backend/src/main/java/com/skapp/community/okrplanner/payload/response/KeyResultResponseDto.java
#	backend/src/main/java/com/skapp/community/okrplanner/payload/response/TeamObjectiveResponseDto.java
#	backend/src/main/java/com/skapp/community/okrplanner/service/TeamObjectiveService.java
#	backend/src/main/java/com/skapp/community/okrplanner/service/impl/TeamObjectiveServiceImpl.java
#	backend/src/test/java/com/skapp/community/okrplanner/controller/v1/TeamObjectiveControllerIntegrationTest.java
@Attigala Attigala marked this pull request as ready for review August 5, 2025 07:50
@Attigala Attigala requested a review from ThinuwanW as a code owner August 5, 2025 07:50
@Attigala Attigala marked this pull request as draft August 5, 2025 07:53
brp-imalsha and others added 7 commits August 6, 2025 15:28
…e-team-objectives

# Conflicts:
#	backend/src/main/java/com/skapp/community/okrplanner/constant/OkrMessageConstant.java
#	backend/src/main/resources/community/db/changelog/db.changelog.yml
#	backend/src/main/resources/community/messages/okr-messages.properties
@sonarqubecloud
Copy link

…e-team-objectives

# Conflicts:
#	backend/src/main/java/com/skapp/community/okrplanner/constant/OkrMessageConstant.java
#	backend/src/main/resources/community/db/changelog/db.changelog.yml
#	backend/src/main/resources/community/messages/okr-messages.properties
#	frontend/middleware.ts
#	frontend/src/community/common/utils/data/routes.ts
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants