-
Notifications
You must be signed in to change notification settings - Fork 569
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: introduce Identity UI #19533
feat: introduce Identity UI #19533
Conversation
Tasklist Test Results146 files ±0 146 suites ±0 1h 33m 11s ⏱️ - 2m 26s Results for commit 72390fb. ± Comparison against base commit 9e5f5fb. This pull request removes 13 and adds 13 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
dist/src/main/java/io/camunda/identity/controllers/IdentityIndexController.java
Outdated
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.
Nice work, looks good to me. I only have one question that I've put in place.
@@ -53,6 +54,8 @@ public void initialize(final ConfigurableApplicationContext context) { | |||
propertyMap.put(CAMUNDA_WEBAPPS_DEFAULT_APP_PROPERTY, TASKLIST.getId()); | |||
propertyMap.put( | |||
SERVER_SERVLET_SESSION_COOKIE_NAME_PROPERTY, TasklistURIs.COOKIE_JSESSIONID); | |||
} else if (activeProfiles.contains(IDENTITY.getId())) { |
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.
Does this logic means that when identity is set in active profiles as the last one, then the default ui is going to be identity's ui?
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.
Not quite, it means that if Identity is present in the active profiles and no other web app profile (operate/tasklist) is set, then Identity will be the default UI (visiting localhost:8080
or localhost:8080/identity
will work). If another webapp profile is there then Identity will be only found on localhost:8080/identity
<meta charset="UTF-8" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>Camunda Indentity</title> | ||
<title>Camunda Identity</title> |
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.
+1 for fixing typo.
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.
We will have another opportunity for including easter eggs 🙈
57b28cf
to
0774bce
Compare
Just wanted to update you @maryarm and @steff46 after your reviews. I spotted an issue when trying to start for example just the broker with the As we have the |
Description
This PR introduces the Identity UI (webjar), it can be served by using the
identity
profile.