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

ui: fix root switch #2110

Merged
merged 1 commit into from
Nov 29, 2024
Merged

ui: fix root switch #2110

merged 1 commit into from
Nov 29, 2024

Conversation

SamuelPull
Copy link
Collaborator

Checklist

  • I have followed (at least) the PR section of the contributing guide.
  • I fixed all necessary PR warnings
  • The commit history is clean
  • The E2E tests are passing
  • If possible, the issue has been divided into more subtasks
  • I did a self review before requesting a review from another team member

Description

How to test

Exactly like #2109, but it should work as expected.

Closes #2109

Copy link

Warnings
⚠️ No CHANGELOG added.
⚠️ There were changes in the frontend, but no E2E-test was added or modified!

Generated by 🚫 dangerJS against 2417e0a

@openkfwCI
Copy link

openkfwCI commented Nov 28, 2024

NotesTime
Note for Reviewer: E2E tests on pipeline 61334 on remote server failedThu, 28 Nov 2024 14:05:43 +0000

Generated by E2E-Test

@SamuelPull SamuelPull merged commit 9181b21 into main Nov 29, 2024
30 checks passed
@SamuelPull SamuelPull deleted the 2109-ui-perms branch November 29, 2024 08:43
galethil added a commit that referenced this pull request Dec 17, 2024
* Patch critical vulnerability (#2098)

* 2097: New TruBudget background (#2099)

* ci: Bump audit versions (#2101)

* api:upgrade fastify (#2102)

* 2079 Project Description field (#2085)

Co-authored-by: Peter Molnar <p.molnar@accenture.com>

* 2083 env vars generator (#2100)

* refactor workflow action styles

* api: refresh and access token configuration

* reverted some changes

* mr comment

* 2107: More descriptive Add buttons (#2108)

* more descriptive buttons

* frontend:New tour prototype

* ui: fix root switch (#2110)

* frontend:Tour with extended functionality

* frontend:Add translations

* frontend:Add timeout clearing for useEffect

* frontend:Add timeout clearing for useEffect

* frontend:Add missing translations

* frontend:Change styles to em

* frontend:Change styles to rem

* fixed vulnerabilities

* few more vulnerabilities fixed

* bump to v2.18.0 (#2118)

Co-authored-by: Peter Baus <peter.baus@accenture.com>

---------

Co-authored-by: SamuelPull <samuel.pull@accenture.com>
Co-authored-by: pmolnar-dev <55734106+pmolnar-dev@users.noreply.github.com>
Co-authored-by: Peter Molnar <p.molnar@accenture.com>
Co-authored-by: MartinJurcoGlina <martin.jurcoglina@accenture.com>
Co-authored-by: Peter Baus <peter.baus@accenture.com>
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.

ui: Permissions not loaded correctly when switching users
3 participants