News server should be pre-populated in Account Wizard after running auto-subscribe URLs.

VERIFIED FIXED in mozilla1.0.1

Status

SeaMonkey
MailNews: Account Configuration
VERIFIED FIXED
16 years ago
14 years ago

People

(Reporter: stephend@netscape.com (gone - use stephen.donner@gmail.com instead), Assigned: Navin Gupta)

Tracking

({regression})

Trunk
mozilla1.0.1
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: custrtm+ [adt2 rtm] [ETA 06/20], URL)

Attachments

(2 attachments)

Build ID: 2002-04-10, all OSs.

Summary: news://news.mozilla.org/netscape.public.mozilla.mail-news

Steps to Reproduce:

1. With no previous news accounts set up, enter
news://news.mozilla.org/netscape.public.mozilla.mail-news into the browser's url
field.
2. Type in your username/email, hit Next.
3. Look at the Server name box on the next screen.

Expected Results:

It should say 'news.mozilla.org'

Actual Results:

It's blank.  The reason it should be filled is this is an auto-subscribe URL.
Keywords: regression
Summary: News server should be pre-populated in Account Wizard after running autosubscribe URLs. → News server should be pre-populated in Account Wizard after running auto-subscribe URLs.
This is a regression that occured as a result of the new <wizard> widget.  It's
not major, but it makes auto-subscribe a whole lot nicer.

Comment 2

16 years ago
marking nsbeta1- per mail triage.
Keywords: nsbeta1 → nsbeta1-
*** Bug 147695 has been marked as a duplicate of this bug. ***

Comment 4

16 years ago
Renominate for [adt rtm]. This breaks CCK's NNTP pre-configuration.
Blocks: 96876, 144547
Keywords: nsbeta1- → nsbeta1
Whiteboard: custrtm+

Updated

16 years ago
Blocks: 139887

Comment 5

16 years ago
Discussed in Mail News bug meeting.  Decided to minus this bug.
Keywords: nsbeta1 → nsbeta1-

Updated

16 years ago
Keywords: nsbeta1- → nsbeta1

Comment 6

16 years ago
Tao- can you confirm that this bug directly impacts feature we are building for
the road runner project? We need to have more discussions before we minus this-
the impact is potentially 1 MM users to Netscape.

Comment 7

16 years ago
Per Jeff: "From an enterprise standpoint this is an ADT1 issue under the
'requirement on PRD' and 'major loss of functionality' criteria.". This
regression breaks spec-ed feature in PRD.

Comment 8

16 years ago
Desired/correct functionality for customized clients
-Default unchecked

In New/Fresh Install:
-  If a user leaves this unchecked, the homepage is left as is in the prefs
file. Default distributor home url (as set by cck tool) remains unchanged.
- If a user check this box, the home page is changed to home.netscape.com
address default to 7.0.

Acceptable senario for established End User with new custom install:
- If a user leaves this unchecked, the homepage is left as set by default
distributor.

IDEAL in this case for established End user with new custom install:
- User is presented with two check boxes, both default unchecked. NOTE: User can
leave unchecked or check one, but not both.

Box 1- Make my homepage xxxx (text set by distributor)
Box 2- Make Netscape.com my homepage

- If a user checks box 1, homepage is changed to url provided by distributor.
- If a user checks box 2, homepage is changed to netscape.com
- If user leaves both unchecked, homepage retained from old profile.

I realize this last one is an enhancement of a feature. I'd like to see what is
involved to make the ones above happen for RTM.

Comment 9

16 years ago
opps wrong bug. please ignore this comment- I had too many windows open at once.

Comment 10

16 years ago
reassigning to naving.
Assignee: racham → naving
Keywords: nsbeta1 → nsbeta1+
Whiteboard: custrtm+ → custrtm+ [adt2 rtm]
(Assignee)

Comment 11

16 years ago
Created attachment 87608 [details] [diff] [review]
proposed fix

The fix is to assign the hostname value if we have one in the server page. I
had to	initialize that ispData to null so that we can initialize pageData with
acct information from invalid acct.
(Assignee)

Comment 12

16 years ago
bhuvan, Can I get r=? thx.

Comment 13

16 years ago
Hi, Naving: thanks for looking into this bug.

Hi, Bom-Shik: would you please verify that the patch also fix CCK's news server
setting problem? thx!

Updated

16 years ago
Attachment #87608 - Flags: review+

Comment 14

16 years ago
Comment on attachment 87608 [details] [diff] [review]
proposed fix

r=bhuvan
(Assignee)

Comment 15

16 years ago
David, can you sr ? thx.
Status: NEW → ASSIGNED

Comment 16

16 years ago
Comment on attachment 87608 [details] [diff] [review]
proposed fix

I thought js variables were automatically initialized to null. The other change
looks fine, however, so sr=bienvenu.
bienvenu - see http://people.netscape.com/stephend/js.html for a null vs.
undefined testcase I wrote

Comment 18

16 years ago
Navin, 

Also, I think you should add a try catch block around the new code so that cases
where there is no pageData for newsserver will not halt the wizard progress.

Bhuvan
(Assignee)

Comment 19

16 years ago
I checked if there is no newsserver it throws the exception saying newsserver
has no properties but doesn't stall the wizard. 
(Assignee)

Comment 20

16 years ago
fixed on trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Keywords: adt1.0.1
Resolution: --- → FIXED

Comment 21

16 years ago
Get on driver's radar, too.
Keywords: mozilla1.0.1

Comment 22

16 years ago
Let's get this verified on the trunk.  blee, can you make sure this solves the
configuration tool's problems?

Comment 23

16 years ago
Trunk build 2002-06-17: WinMe, Linux RH 7.1, Mac 10.1.3
Verified Fixed, the news server was displayed in the Account Wizard using the 
following test scenarios:

1. New profile, autoscribe: ok
2. New profile, configure for a Mail account, then autosubscribe: ok
3. Migrate a profile which is configured for mail and the "news" server. 
Autosubscribe to news.mozilla.org: ok
4. Migrate a profile which is configured for news.mozilla.org, remove 
news.mozilla.org, exit/restart, autosubscribe: ok 
but it takes 15 minutes on my system for the Account Wizard to appear and 3-5 
minutes on Esther's WinXP system. I will log this issue in a different bug. 
Status: RESOLVED → VERIFIED

Comment 24

16 years ago
I tried this out by pointing 6/17 CCK to the blob installer of 06-17-12 trunk 
bld, but NNTP server setting still does not not show up in the custom bld.

Comment 25

16 years ago
Created attachment 88150 [details]
newsaccount.rdf created by CCK 6.2

Comment 26

16 years ago
Per tao's suggestion, I copied newsaccount.rdf (created by CCK 6.2) in 
/default/isp(/us) of 06-17-12 trunk bld and it displayed the custom content 
correctly. That verifies this bug is fixed. Sorry about the confusion. 
The .rdf file is attached here.

Comment 27

16 years ago
It probably doesn't skip the wizardpanels (as requested in the rdf file) still.
Skipping panels meaning jumping from identity page to Congratulations/Done page.
  I have fixed this part in the patch for bug 144562.

Comment 28

16 years ago
Comment on attachment 87608 [details] [diff] [review]
proposed fix

has sr=bienvenu
Attachment #87608 - Flags: superreview+

Updated

16 years ago
Attachment #87608 - Flags: approval+

Comment 29

16 years ago
please checkin to the 1.0.1 branch. once there, remove the "mozilla1.0.1+"
keyword and add the "fixed1.0.1" keyword.
Keywords: mozilla1.0.1 → mozilla1.0.1+

Comment 30

16 years ago
adt1.0.1+ (on ADT's behalf) approval for checkin to the 1.0 branch. pls check
this in asap, then add the "fixed1.0.1" keyword.
Keywords: adt1.0.1 → adt1.0.1+
Whiteboard: custrtm+ [adt2 rtm] → custrtm+ [adt2 rtm] [ETA 06/20]
Target Milestone: --- → mozilla1.0.1
(Assignee)

Comment 31

16 years ago
fixed on branch
Keywords: mozilla1.0.1+ → fixed1.0.1

Comment 32

16 years ago
Branch build 2002-06-21: WinMe, Linux RH 7.1, Mac 9.1
verified1.0.1
Keywords: fixed1.0.1 → verified1.0.1
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.