Skip to content

Add factor delete confirmation#348

Merged
j-or merged 4 commits intomainfrom
add-factor-delete-confirmation
Sep 24, 2025
Merged

Add factor delete confirmation#348
j-or merged 4 commits intomainfrom
add-factor-delete-confirmation

Conversation

@j-or
Copy link
Copy Markdown
Contributor

@j-or j-or commented Sep 23, 2025

Screenshot 2025-09-22 at 15 56 59 Screenshot 2025-09-22 at 15 57 07 Screenshot 2025-09-22 at 15 57 14

@j-or j-or requested a review from langdal September 23, 2025 10:19
@langdal
Copy link
Copy Markdown
Member

langdal commented Sep 23, 2025

Det var hurtigt!

Copy link
Copy Markdown
Member

@langdal langdal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der er lige en test der ikke fungerer, men ellers ser det godt ud :)

@j-or
Copy link
Copy Markdown
Contributor Author

j-or commented Sep 23, 2025

Der er lige en test der ikke fungerer, men ellers ser det godt ud :)

Ja... synes ikke det ligner noget jeg har pillet ved. Jeg ser på det.

@langdal
Copy link
Copy Markdown
Member

langdal commented Sep 23, 2025

Jeg kan godt se det ikke er nyt i denne branch - det er nok kommet af at jeg har opdateret dependencies. Jeg kan godt se på det.

@j-or j-or merged commit ae9aa47 into main Sep 24, 2025
1 check passed
@j-or j-or deleted the add-factor-delete-confirmation branch September 24, 2025 10:46
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.

2 participants