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

fix(projectsADM): clear cache after changing project (DEV-239) #1943

merged 4 commits into from Nov 19, 2021


Copy link

@mpro7 mpro7 commented Nov 18, 2021

resolves DEV-239 + rebase last commit message

…lementation (DEV-160) (#1932)

* refactor group create payload

* separate group route value object

* add group iri value object

* use project IRI in group create patload + add group value object error messages &  test

* updated groups value objects and tests

* add value objects to groups update payload

* move optional params to value objects + replace either by zio.Validation

* refactor(list): replace Either with zio.Validation + move option to value objects

* refactor projectIri value object

* fix node creation

* add validation to payloads creation in lists and groups

* remove doubled checks and related tests

* disable new list route + separate NodeApiRequestADM to root & child

* list route factory + new list route implementation

* refactor list naming

* make parentNodeIri required for ListChildNodeCreatePayloadADM

* fix tests

* refactor lists get methods

* update lists docs

* fix update single lists properties + groups cleanup

* fix client-test-data

* bring back GET methods

* fix file extension

* fix testdata
@mpro7 mpro7 self-assigned this Nov 18, 2021
@mpro7 mpro7 requested a review from subotic as a code owner November 18, 2021 11:21
@mpro7 mpro7 changed the title Wip/dev 239 fix(userADM): disable catching for GET single user methods (DEV-239) Nov 18, 2021
Copy link

sonarcloud bot commented Nov 19, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mpro7 mpro7 changed the title fix(userADM): disable catching for GET single user methods (DEV-239) fix(projectsADM): clear cache after changing project (DEV-239) Nov 19, 2021
@mpro7 mpro7 merged commit 17c5c09 into main Nov 19, 2021
@mpro7 mpro7 deleted the wip/DEV-239 branch November 19, 2021 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants