Skip to content

Commit

Permalink
web: fix labels on group view page (#7677)
Browse files Browse the repository at this point in the history
* web: break circular dependency between AKElement & Interface.

This commit changes the way the root node of the web application shell is
discovered by child components, such that the base class shared by both
no longer results in a circular dependency between the two models.

I've run this in isolation and have seen no failures of discovery; the identity
token exists as soon as the Interface is constructed and is found by every item
on the page.

* web: fix broken typescript references

This built... and then it didn't?  Anyway, the current fix is to
provide type information the AkInterface for the data that consumers
require.

* web: rollback dependabot's upgrade of context

The most frustrating part of this is that I RAN THIS, dammit, with the updated
context and the current Wizard, and it finished the End-to-End tests without
complaint.

* web: fix labels on group view page

This is a wild bug, because what caused it and how it manifested are seemingly
unrelated as to be hallcinatory.
  • Loading branch information
kensternberg-authentik committed Nov 22, 2023
1 parent 30f9d6b commit 255e143
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion web/src/admin/groups/GroupViewPage.ts
Expand Up @@ -118,7 +118,7 @@ export class GroupViewPage extends AKElement {
<div class="pf-c-description-list__text">
<ak-status-label
type="warning"
?good${this.group.isSuperuser}
?good=${this.group.isSuperuser}
></ak-status-label>
</div>
</dd>
Expand Down

0 comments on commit 255e143

Please sign in to comment.