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

Feat/delete from layout config #42

Merged
merged 5 commits into from
Oct 17, 2024
Merged

Conversation

malmen237
Copy link

What does this do?

This adds a trash bin to the layout-config modal, when pressed the selected layout is deleted.
Made it not possible when production is running.
Screenshot 2024-10-16 at 13 17 00

Added confirmation when delete is done and layout is set to default:
Screenshot 2024-10-16 at 16 28 16

} catch (error) {
console.log(error);
return new NextResponse(
`Error occurred while posting to DB! Error: ${error}`,

Choose a reason for hiding this comment

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

--> Error occurred while deleting from DB! Error: ${error}

Copy link

@Saelmala Saelmala left a comment

Choose a reason for hiding this comment

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

LGTM! Just left a small comment on changing the error message from 'post to db' to 'delete from db'

@malmen237 malmen237 merged commit 4dc597e into main Oct 17, 2024
4 checks passed
@malmen237 malmen237 deleted the feat/delete-from-layout-config branch October 17, 2024 12:51
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