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
Feature/2584/adjust default number of displayed labels #3046
Feature/2584/adjust default number of displayed labels #3046
Conversation
...zation/app/codeCharta/state/store/appSettings/amountOfTopLabels/amountOfTopLabels.actions.ts
Outdated
Show resolved
Hide resolved
… top labels, event handling is still WIP #2584
...ation/app/codeCharta/state/store/appSettings/amountOfTopLabels/amountOfTopLabels.selector.ts
Outdated
Show resolved
Hide resolved
...ation/app/codeCharta/state/store/appSettings/amountOfTopLabels/amountOfTopLabels.selector.ts
Outdated
Show resolved
Hide resolved
… of github.com:MaibornWolff/codecharta into feature/2584/adjust-default-number-of-displayed-labels
@shaman-apprentice What did I do wrong in the tests for the effect? The selectors ignore my mocking implementation. |
...on/app/codeCharta/state/effects/updateVisibleTopLabels/updateVisibleTopLabels.effect.spec.ts
Outdated
Show resolved
Hide resolved
...ization/app/codeCharta/state/effects/updateVisibleTopLabels/updateVisibleTopLabels.effect.ts
Outdated
Show resolved
Hide resolved
...on/app/codeCharta/state/effects/updateVisibleTopLabels/updateVisibleTopLabels.effect.spec.ts
Outdated
Show resolved
Hide resolved
…er-of-displayed-labels' into feature/2584/adjust-default-number-of-displayed-labels
...on/app/codeCharta/state/effects/updateVisibleTopLabels/updateVisibleTopLabels.effect.spec.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…er-of-displayed-labels' into feature/2584/adjust-default-number-of-displayed-labels
I would like to wait until Friday because I have a conversation with Torsten and there are still some questions open. |
…efault-number-of-displayed-labels
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
Adjust default number of displayed labels
Closes: #2584
Description