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

Check import ics for VTODOs and filter Calendars accordingly #2572

Open
Matthias84 opened this issue Sep 9, 2020 · 5 comments
Open

Check import ics for VTODOs and filter Calendars accordingly #2572

Matthias84 opened this issue Sep 9, 2020 · 5 comments
Labels
1. to develop Accepted and waiting to be taken care of bug

Comments

@Matthias84
Copy link

Steps to reproduce

  1. Split big .ics files of a old NC export (to workaround Importing bigger .ICS files only imports part of it #2570)
  2. Start importing the first files, which runs fine
  3. Notice that the importer doesn't list the desired calendar for the last two files anymore

I compared the files, but they look fine so far (VCALENDAR surrounding items, VEVENT and VTODO items). I also tried the first files, which looks good. Changing filename doesn't solve the issue. Reimporting the original big file, doesn't also work anymore. Logout and new session doesn't helped as well.

Expected behaviour

Like all the other files, I wan't to import it to the same calendar so far.

Actual behaviour

I can't pick the right (existing) calendar, to add more items to this calendar.
(no browser console messages appear, explaining why the item is removed from the list)
Bildschirmfoto vom 2020-09-09 21-01-51

Calendar app

Calendar app version: 2.0.4

CalDAV-clients used: -

Client configuration

Browser: Firefox 80

Operating system: Linux Mint 20

Server configuration

Operating system: CentOS

Web server: Nginx

Database: MariaDB

PHP version: (e.g. 7.0.3)

Nextcloud Version: 19.0.2

Updated from an older installed version or fresh install: fresh

@Matthias84 Matthias84 added 0. to triage Pending approval or rejection bug labels Sep 9, 2020
@georgehrke
Copy link
Member

I compared the files, but they look fine so far (VCALENDAR surrounding items, VEVENT and VTODO items).

If you check the database (oc_calendars), does the component type field for the calendar contain VTODO or just VEVENT?
It only offers calendars if all contained component types are supported by the calendar.

@Matthias84
Copy link
Author

Thanks for the hint! I used your tip from #2571 (comment) to force a detection of both types and it worked fine.

How about to explain the user this auto-behaviour? (e.g. disable list item / textfield if detection was triggered / let user choose the supported types manually before import, ...)

@tdoerschel
Copy link

tdoerschel commented Oct 28, 2021

Hi,
i also stumbled over this issue:
any new calender that is created via the webinterface has no VTODO (no task list).
Therefore, if I try to import from ics file that has both tasks and events, I cannot import it to the existing (within-nextcloud-created) calender.

Solution:
When creating a new calender from webinterface, also create a task list for it (VEVENT and VTODO).

P.S. When creating a new task list, a new calender is automatially generated. So, it should just work the other way as well.
However, I still cannot import a ics file to a new calender from tasks (although it should have both fields)

All tested on NC 22.20, calender 2.3.4 and tasks 0.14.2 with PHP8.0-FPM, mysql 10.3.31

@tcitworld
Copy link
Member

any new calender that is created via the webinterface has no VTODO (no task list).

You can create it that way, but not from the import modal indeed.
image

@miaulalala miaulalala changed the title Importer hides one existing calendar for some .ics files? Check import ics for VTODOs and filter Calendars accordingly Jul 1, 2024
@miaulalala
Copy link
Contributor

Suggested solution

Check a file that is being imported before we start the process for the different types. If the file contains VTODOs only show calendars that support VTODO and also let the user know that this is the case so this information isn't inaccessible.

I would also like to see the supported component types somewhere, so users know what types are supported for each calendar.

@miaulalala miaulalala added 1. to develop Accepted and waiting to be taken care of and removed 0. to triage Pending approval or rejection labels Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug
Projects
None yet
Development

No branches or pull requests

5 participants