Calendars imported/created by Migration Wizard are not visible by default

RESOLVED FIXED

Status

Calendar
Import and Export
--
minor
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Stefan Sitter, Assigned: Joey Minta)

Tracking

Trunk
x86
Windows 2000

Details

Attachments

(1 attachment)

1.75 KB, patch
Matthew (lilmatt) Willis
: first-review+
Michiel van Leeuwen (email: mvl+moz@)
: second-review+
Michiel van Leeuwen (email: mvl+moz@)
: ui-review+
Details | Diff | Splinter Review
(Reporter)

Description

11 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/20070107 Calendar/0.4a1

The Migration Wizard successfully imported the existing calendars from Sunbird 0.2 profile but unfortunately this calendars are not visible by default. Users might think that something went wrong although the wizard reported success if the events are not visible.

Steps to Reproduce:
1. Create a new profile using Sunbird 0.2
2. Create test calendars and test data using Sunbird 0.2
3. Open profile with current Sunbird build
4. Let the Migration Wizard import the Sunbird 0.2 calendars
5. Check the events

Actual Results:
No events are displayed in calendar view or unifinder. You have to go to the Calendars tab and manually enable every new calendar to make the events visible.

Expected Results:
The new calendars should be enabled/visible by default.
(Assignee)

Comment 1

11 years ago
Created attachment 250762 [details] [diff] [review]
add to composite

Patch adds any created calendars to the composite calendar, so they'll be checked.  Normally, the observer would handle this, but migration happens so early in the startup cycle that the observer doesn't exist yet.
Assignee: nobody → jminta
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #250762 - Flags: ui-review?(mvl)
Attachment #250762 - Flags: second-review?(mvl)
Attachment #250762 - Flags: first-review?(lilmatt)
Comment on attachment 250762 [details] [diff] [review]
add to composite

r=lilmatt
Attachment #250762 - Flags: first-review?(lilmatt) → first-review+
Comment on attachment 250762 [details] [diff] [review]
add to composite

Yes, we really need to do this.
ui-review+r2=mvl
Attachment #250762 - Flags: ui-review?(mvl)
Attachment #250762 - Flags: ui-review+
Attachment #250762 - Flags: second-review?(mvl)
Attachment #250762 - Flags: second-review+
(Assignee)

Comment 4

11 years ago
patch checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.