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

"More details" of fresh event is broken #5840

Closed
ChristophWurst opened this issue Mar 8, 2024 · 3 comments · Fixed by #5866
Closed

"More details" of fresh event is broken #5840

ChristophWurst opened this issue Mar 8, 2024 · 3 comments · Fixed by #5866
Assignees
Labels
3. to review Waiting for reviews bug regression Regression of a previous working feature skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills
Milestone

Comments

@ChristophWurst
Copy link
Member

Steps to reproduce

  1. Click into the calendar
  2. Enter an event title in the simple editor
  3. Click Save
  4. Click More details

Expected behavior

Event shows in the sidebar

Actual behaviour

Bildschirmfoto vom 2024-03-08 08-05-17

URL is /apps/calendar/

Calendar app version

4.7

CalDAV-clients used

No response

Browser

No response

Client operating system

No response

Server operating system

No response

Web server

None

Database engine version

None

PHP engine version

None

Nextcloud version

No response

Updated from an older installed version or fresh install

None

List of activated apps

No response

Nextcloud configuration

No response

Web server error log

No response

Log file

No response

Browser log

No response

Additional info

No response

@ChristophWurst ChristophWurst added 1. to develop Accepted and waiting to be taken care of bug labels Mar 8, 2024
@ChristophWurst ChristophWurst added this to the v4.7.0 milestone Mar 8, 2024
@ChristophWurst ChristophWurst added skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills regression Regression of a previous working feature labels Mar 8, 2024
@ChristophWurst
Copy link
Member Author

Regression possibly introduced with #3543

@st3iny st3iny added the 2. developing Work in progress label Mar 20, 2024
@st3iny st3iny moved this from 📄 To do to 🏗️ In progress in 💌 📅 👥 Groupware team Mar 20, 2024
@st3iny st3iny removed the 1. to develop Accepted and waiting to be taken care of label Mar 20, 2024
st3iny added a commit that referenced this issue Mar 20, 2024
@st3iny st3iny added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 20, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Mar 21, 2024
@executed
Copy link

@ChristophWurst @st3iny

STR:

  1. Simple Editor is disabled
  2. There's at least one event on daily view
  3. User clicks on an empty time slot
  4. User does not fill anything in the revealed sidebar (event is empty)
  5. User clicks on other pre-existing event;
  6. Sidebar for other pre-existing event reveals;
  7. Empty event that user just jumped focus from gets saved as empty
  8. User observes title "Untitled event"

Looks like a bug to me as after heavy daily planning sessions you get to clean up bunch of those empty events.
Do you think it was caused by this PR?
Perhaps there's another issue where this bug is being tracked?

@SebastianKrupinski
Copy link
Contributor

@executed

Tested and Confirmed your issue.

I've started a new ticket for this as not to hijack this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug regression Regression of a previous working feature skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills
Projects
Development

Successfully merging a pull request may close this issue.

4 participants