Skip to content

Commit

Permalink
2751 remove duplicate ids (#2754)
Browse files Browse the repository at this point in the history
* Dom ID housekeeping

* Update CHANGELOG
  • Loading branch information
rorymckinley authored Dec 11, 2024
1 parent b78b5c7 commit 5effcb3
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ and this project adheres to

### Fixed

- Delete user modal no longer uses the same id as the underlying user record.
[#2751](https://github.com/OpenFn/lightning/issues/2751)

## [v2.10.6] - 2024-12-10

### Added
Expand Down
4 changes: 2 additions & 2 deletions lib/lightning_web/live/components/user_deletion_modal.ex
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ defmodule LightningWeb.Components.UserDeletionModal do
def render(%{delete_now?: true, has_activity_in_projects?: true} = assigns) do
~H"""
<div>
<.modal id={"user-#{@id}"} width="max-w-md" show={true}>
<.modal id={"user-#{@id}-delete-modal"} width="max-w-md" show={true}>
<:title>
<div class="flex justify-between">
<span class="font-bold">
Expand Down Expand Up @@ -129,7 +129,7 @@ defmodule LightningWeb.Components.UserDeletionModal do
def render(assigns) do
~H"""
<div>
<.modal id={"user-#{@id}"} show={true} width="max-w-md">
<.modal id={"user-#{@id}-delete-modal"} show={true} width="max-w-md">
<:title>
<div class="flex justify-between">
<span class="font-bold">
Expand Down

0 comments on commit 5effcb3

Please sign in to comment.