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

Delete modal runs action too early #10754

Closed
morhi opened this issue Sep 3, 2024 · 0 comments · Fixed by #10755
Closed

Delete modal runs action too early #10754

morhi opened this issue Sep 3, 2024 · 0 comments · Fixed by #10755
Labels

Comments

@morhi
Copy link
Contributor

morhi commented Sep 3, 2024

Bug description

When deleting entries and there are entries with descendants the modal would ask if you want to delete or detach the descendants (localizations). However, the modal already sends the request when clicking the button whether to delete or detach the descendants and not when clicking the actual red delete button.

Screen.Recording.2024-09-03.at.15.03.34.mov

How to reproduce

Delete many entries having descendants and press the delete option for descendants. The action will run immediately instead of after clicking the red delete button.

Logs

No response

Environment

Environment
Application Name: Statamic
Laravel Version: 11.21.0
PHP Version: 8.2.20
Composer Version: 2.7.7
Environment: local
Debug Mode: ENABLED
URL: localhost
Maintenance Mode: OFF
Timezone: UTC
Locale: de

Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: NOT CACHED

Drivers
Broadcasting: log
Cache: file
Database: mysql
Logs: stack / single
Mail: smtp
Queue: sync
Session: file

Statamic
Addons: 19
Sites: 2 (Deutsch, Französisch)
Stache Watcher: Enabled
Static Caching: Disabled
Version: 5.23.0 PRO

Statamic Addons
doefom/statamic-export: 0.4.0
duncanmcclean/simple-commerce: 7.4.5
jonassiewertsen/statamic-livewire: 3.6.0
rias/statamic-redirect: 3.7.2
thoughtco/statamic-cache-tracker: 0.8.2
...

Installation

Fresh statamic/statamic site via CLI

Additional details

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants