deck icon indicating copy to clipboard operation
deck copied to clipboard

delete card in modal view: modal should close

Open alexanderdd opened this issue 1 year ago • 2 comments

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

To Reproduce Steps to reproduce the behavior:

  1. open a card (modal view)
  2. in top right ...-menu, click delete
  3. modal is still open but just blank

Expected behavior Modal should close

NC 30.0.0 RC2 c.nc.com (can't check app version)

alexanderdd avatar Aug 28 '24 15:08 alexanderdd

@nextcloud/office can you please triage this? I just reproduced it on c.nc.com

alexanderdd avatar Oct 15 '24 07:10 alexanderdd

Here's a brief video of the reproduction. https://github.com/user-attachments/assets/0df6a62f-b610-4222-bc4c-11dbdef223a4

Seems like an easy thing to add. So marking as good first issue.

max-nextcloud avatar Oct 15 '24 14:10 max-nextcloud

This was a great first issue for me :) Please see my PR above or here: https://github.com/nextcloud/deck/pull/6617

ludij avatar Dec 19 '24 00:12 ludij

Thanks a lot for your contribution @ludij

Looking forward to any further, let us know if you have any questions. In case you are interested, we have a community chat to stay in contact with active contributors and have a more direct channel for questions and discussions. I can invite you, in case you're interested just drop me a mail to the address in my github profile.

juliusknorr avatar Dec 19 '24 22:12 juliusknorr

@juliusknorr is this fix already in production on cloud.nextcloud.com? (30.0.5 RC1 / cant check deck version)

cause there it is not fixed yet

alexanderdd avatar Jan 14 '25 05:01 alexanderdd

https://apps.nextcloud.com/apps/deck/releases?platform=30#30 says the last release happened in November. So the fix has not been released yet.

max-nextcloud avatar Jan 14 '25 08:01 max-nextcloud

@luka-nextcloud @grnd-alt Can you align on building one? Maybe we should have a fixed schedule for the releases (we can discuss this in the team call next week)

juliusknorr avatar Jan 14 '25 09:01 juliusknorr

can confirm it is fixed now on cloud.nextcloud.com - Thank you!!

alexanderdd avatar Jan 17 '25 16:01 alexanderdd