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.

Sign up for GitHub

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

Jump to bottom

Shell 101 fix test #281

Merged
merged 4 commits into from
Jul 10, 2023
Merged

Shell 101 fix test #281

merged 4 commits into from
Jul 10, 2023

Conversation

beawar
Copy link
Contributor

@beawar beawar commented Jul 6, 2023

No description provided.

@beawar beawar marked this pull request as ready for review July 7, 2023 12:29
@beawar beawar requested review from a team as code owners July 7, 2023 12:29
@beawar beawar requested review from FrancescoGottardi, AlessioCrestani and eugeniamuff and removed request for a team July 7, 2023 12:29
@beawar beawar merged commit adaecfd into SHELL-121-whitelabel-support-custom-logout Jul 10, 2023
@beawar beawar deleted the SHELL-101-fix-test branch July 10, 2023 10:40
@beawar beawar mentioned this pull request Jul 10, 2023
Merged
beawar added a commit that referenced this pull request Jul 11, 2023
@beawar
feat(whitelabel): redirect to custom logout url when user logout
f9d30b6 
test: add test for logout and improve other tests
refactor: improve load functions
test: refactor test and mock reporting and workers for all tests
refactor: use direct import for reporting and remove index file

refs: SHELL-121 (#280) SHELL-101 (#281)
beawar added a commit that referenced this pull request Aug 18, 2023
@beawar
feat(whitelabel): redirect to custom logout url when user logout
05ca3e7 
test: add test for logout and improve other tests
refactor: improve load functions
test: refactor test and mock reporting and workers for all tests
refactor: use direct import for reporting and remove index file

refs: SHELL-121 (#280) SHELL-101 (#281)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CataldoMazzilli CataldoMazzilli CataldoMazzilli approved these changes

@FrancescoGottardi FrancescoGottardi Awaiting requested review from FrancescoGottardi FrancescoGottardi was automatically assigned from zextras/crabonions

@AlessioCrestani AlessioCrestani Awaiting requested review from AlessioCrestani AlessioCrestani was automatically assigned from zextras/crabonions

@eugeniamuff eugeniamuff Awaiting requested review from eugeniamuff eugeniamuff was automatically assigned from zextras/crabonions

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@beawar @CataldoMazzilli