Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add updateResource endpoint and tests [DEV-4606] #367

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

sownak
Copy link

@sownak sownak commented Dec 10, 2024

  • Add updateResource endpoint
  • Add playwright tests

@ankurdotb
Copy link
Contributor

Signed-off-by: Sownak Roy <[email protected]>
src/controllers/resource.ts Outdated Show resolved Hide resolved
src/controllers/resource.ts Outdated Show resolved Hide resolved
@sownak
Copy link
Author

sownak commented Dec 12, 2024

@DaevMithran All comments are address in this PR

@sownak sownak merged commit e236c5f into feat/updated-endpoints Dec 13, 2024
14 checks passed
@sownak sownak deleted the feat/update-resource branch December 13, 2024 09:09
Eengineer1 pushed a commit that referenced this pull request Dec 17, 2024
* feat: add createResource endpoint

Signed-off-by: Sownak Roy <[email protected]>

* feat: Add updateResource endpoint and tests [DEV-4606] (#367)

* feat: Add updateResource endpoint and tests

* Updated with review comments

Signed-off-by: Sownak Roy <[email protected]>

* updated traits

Signed-off-by: Sownak Roy <[email protected]>

* Updated swagger docs

Signed-off-by: Sownak Roy <[email protected]>

* review comments, and tests restructure

Signed-off-by: Sownak Roy <[email protected]>

---------

Signed-off-by: Sownak Roy <[email protected]>
cheqd-bot bot pushed a commit that referenced this pull request Dec 17, 2024
## [2.3.0-develop.1](2.2.0...2.3.0-develop.1) (2024-12-17)

### Features

* add createResource endpoint [DEV-4605][DEV-4636] ([#366](#366)) ([25b5f59](25b5f59)), closes [#367](#367)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants