Closed Bug 399802 Opened 17 years ago Closed 16 years ago

don't migrate RSS accounts until we have a RSS reader

Categories

(SeaMonkey :: Startup & Profiles, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: mnyromyr, Unassigned)

References

Details

Attachments

(1 file)

When importing a Thunderbird profile which has an RSS account configured, we import that as is. Of course, it doesn't work (and its properties panel even results in a XUL error because it's not there)...
Attached patch The fixSplinter Review
This fixes the problem. If we don't know the type of server, then we skip that server (and any copying) and then we strip out the prefs at the end of the function so that they don't get written into the new prefs.js file.
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #284876 - Flags: review?(neil)
Depends on: 329744
Sorry, but I don't see how this works.
* Where do you remove the rss server from the list of servers pref?
* Where do you remove the account for the rss server?
* Where do you remove the identity for the rss account?
(In reply to comment #2)
> Sorry, but I don't see how this works.
> * Where do you remove the rss server from the list of servers pref?

Where's this pref? I couldn't find it on my current build.

> * Where do you remove the account for the rss server?
> * Where do you remove the identity for the rss account?

ok, I missed these. They will probably require a bit more rework.
I mentioned this on IRC, but can't recall if I caught you Mark.

But is there any chance in storing at least RSS feed data in a "hidden" prefBranch so that when the RSS stuff (that I am hoping to finish) is done we won't need to re-do a migration to make it all work, I can just pull from those hidden prefs with something like,

"SeaMonkey has discovered old RSS accounts from the profile migration, would you like to import them now?"  clicking yes/ok would of course move them straight into the normal account view using whatever code I come up with.  No, would just delete that pref branch.
(In reply to comment #4)
> But is there any chance in storing at least RSS feed data in a "hidden"
> prefBranch so that when the RSS stuff (that I am hoping to finish) is done we
> won't need to re-do a migration to make it all work, I can just pull from those
> hidden prefs with something like,
> "SeaMonkey has discovered old RSS accounts from the profile migration, would
> you like to import them now?"  clicking yes/ok would of course move them
> straight into the normal account view using whatever code I come up with.  No,
> would just delete that pref branch.

It is a possibility. The other option could be to have a menu option "Import RSS accounts" that would be capable of importing from Thunderbird profiles (see Import Bookmarks in the bookmarks mananger, current trunk, for an example, except that only does existing SeaMonkey bookmarks at the moment).
Comment on attachment 284876 [details] [diff] [review]
The fix

This needs more rework than what we've got here.
Attachment #284876 - Flags: review?(neil)
Mark,
Are you still working on this ?
(In reply to comment #7)
> Mark,
> Are you still working on this ?
> 
Err, not at the moment.
Assignee: bugzilla → nobody
Status: ASSIGNED → NEW
We now have a mailnews feed reader... WONFTIX
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: