Skip to content

Reafactor/deal moving v1 to v2 be phase 02 - #1962

Open
YasinduRC wants to merge 3 commits into
developfrom
reafactor/deal-moving-v1-to-v2-BE-phase-02
Open

YasinduRC wants to merge 3 commits into
developfrom
reafactor/deal-moving-v1-to-v2-BE-phase-02

Conversation

@YasinduRC

Copy link
Copy Markdown

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

@YasinduRC YasinduRC self-assigned this Sep 18, 2026
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Unreferenced on develop and superseded by CrmBoardInitDataResponseDto.
Grouped with the v2 deal layer removal so the migration PR stays scoped
to the response shape change.
@YasinduRC
YasinduRC force-pushed the reafactor/deal-moving-v1-to-v2-BE-phase-02 branch from 70836ba to b0ae6db Compare September 18, 2026 05:15
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants