14301 - "Log into this account at startup" doesn't work

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect, P2)

Description

[Ninoschka Baca]

Build 1999091709M1: Win32/NT4
(Linux and Mac. Will check with more detail later)

Overview:

1. Problem: Not seeing the option to verify "Log into this account at startup of
Mail" (Bug#14065?)
a. therefore can't verify if this is selected by default.
b. can't verify that the "Set as Default" button is working.
Expect: This option to be selected by default for the first account created in a
profile.

Identity
2. Problem: The "Email Address" text box accepts "qatest20" as a valid email
address.
Expect: It should check for a valid email address such as [e-mail].

3. Problem: The "Reply-to Address" text box accepts "qatest20" as a valid email
address.
Expect. It should check for a valid email address such as [e-mail].

4. Problem: The "Edit vCard" button does nothing.
Expect: The "Edit vCard" button should launch the Address Book Card for the
current identity.

5. Problem: The "Edit" button does nothing.
Expect: The "Edit" button should open the selected signature file for editing.

POP Server
6. Problem: For the "Check for new messages every # messages" text box enter a
character such as "a" and it appears to accept it.
Expect: A warning similar to "Please enter numeric digits only." should appear.

7. Problem: For the "Ask me before downloading more than # messages." text box
enter a character such as "a" and it appears to accept it.
Expect: A warning similar to "Please enter numeric digits only." should appear.

Comment 1

[Ninoschka Baca]

QA Contact to nbaca.

Comment 2

[lchiang]

Should these be separate bug reports?

Comment 3

[Alec Flett]

Issues fixed so far:
2. and 3. are dupes of #3744, so let's not count them as part of this bug.
4. I've commented out the vcard UI for right now (I'm not sure we're
implementing it for PR1)
5. for PR1, we're not supporting editing of signatures, so I took that out.
6. and 7. we won't have text field filters for PR1, so latering that issue to
after PR1

So only issue #1 is good for PR1. Let's open a seperate bug for #6 and #7

Comment 4

[lchiang]

(target milestone is M11 or M12 - add to beta tracking bug)

Comment 5

[Ninoschka Baca]

Created a separate bug for items #6 and #7 (Bug# 14525)
For this bug I will only be concerned about item #1.

Comment 6

[Alec Flett]

I think the potential changes here are big enough to move to M14

Comment 7

[Phil Peterson]

M15

Comment 8

[selmer (gone)]

Minor UI fixups to M18.  If this is incorrect, please adjust.

Comment 9

[Ninoschka Baca]

Since the UI doesn't expose this functionality, is there a way to edit the prefs 
to test logging into accounts at startup?

Comment 10

[Alec Flett]

I don't believe any of it has been implemented yet, even in the backend (I don't
see an entry in nsIMsgIncomingServer)

Comment 11

[Ninoschka Baca]

Is there a bug that tracks this functionality because I can't find one.

Comment 12

[Alec Flett]

I don't know. Putterman might have one... I thought he was on the CC the whole
time, adding him now.

Comment 13

[esther]

Update:  it looks link the backend is working now (build 2000-06-07-21) on win98 
and is logging into all the IMAP accounts by default.   We need the front end 
now so users can choose which accounts they want to log into.   

Comment 14

[Alec Flett]

actually, for some reason it has started logging into all IMAP accounts that are
expanded on startup. I think this may have come out of seth's
folder-discovery-on-expand work.

Comment 15

[(not reading, please use [e-mail] instead)]

there is a bug already about the "trying to log on to all expanded imap / nntp
accounts".

we call perform expand on any servers that are expanded.

see PerformExpandForAllOpenServers() in msgMail3PaneWindow.js

I added that originally so that we'd update the unread counts for news.
we recently implemented PerformExpand() for imap servers.

perhaps we should only call perform expand on a server (at startup) if 
we have remembered its password.

I'm open to any suggestions, and I'll gladly fix the problem.

Comment 16

[(not reading, please use [e-mail] instead)]

how about this:  for any account that has "log into this account" set, we call
PerformBiff()

Comment 17

[scottputterman]

I think we want to do a Get Messages when mail first comes up. I think a lot of 
making this work would be part of making Get Messages for multiple accounts 
work. So whether it's GetMessages or PerformBiff which I guess always forces a 
GetMessages nowadays we should probably do something. Currently biff doesn't go 
off if we aren't logged in.

Comment 18

[scottputterman]

adding jglick to cc list.

Do users expect to have this happen automatically or would most users log into 
each account separately?

Comment 19

[jglick]

I think there are two things going on here.

1. Account Settings - Server(POP or IMAP)would have:
"Log onto this account at startup of Mail" check box.  The would be turned on 
for each new account by default. That means as the default, users would get a 
password dialog for each account at startup of Mail.

http://gooey/client/5.0/specs/mail/accountSetup/images/AcctServerPop1.gif

2. Preferences:
We had also discussed having a preference for the behavior of the "Get Msg" 
button.  The default being get message for the currently selected account only 
vs get messages for all account you are currently logged into.

http://gooey/client/5.0/specs/mail/Prefs/images/PrefsMnN1.gif

Is #2 not valid anymore?

As for what users expect, so far in testing, a lot of users just expect their 
new messages to come to them automatically. "They should just be there." Most 
folks haven't been real clear on what the "Get Msgs" button is for.  "If i have 
new mail, shouldn't it just be in my Inbox?".

Comment 20

[Alec Flett]

massive reassign of account manager bugs -> sspitzer
please feel free to put me back on the CC if you have any questions/comments

Comment 21

[Ninoschka Baca]

Adding mail3 as keyword. "Log into this account at startup" is important for 
usability reasons. Depending on how its implemented it may make it easier for 
user's to see thier new messages. 

Comment 22

[scottputterman]

moving to mozilla0.9 and marking nsbeta1+

Comment 23

[scottputterman]

moving to mozilla0.8 milestone.

Comment 24

[(not reading, please use [e-mail] instead)]

working on the UI and back end elements of this per server pref for this now.
after that, I'll work on actually logging into accounts on startup.

jglick, is the ui for this in the spec anywhere?

Comment 25

[(not reading, please use [e-mail] instead)]

removing lisa
adding racham

Comment 26

[(not reading, please use [e-mail] instead)]

ui and back end (to support the account manager ui) is ready to land.
I'll try to land it tomorrow.

Comment 27

[(not reading, please use [e-mail] instead)]

*** This bug has been marked as a duplicate of 62731 ***

Comment 28

[(not reading, please use [e-mail] instead)]

whoops, sorry.

Comment 29

[(not reading, please use [e-mail] instead)]

re-accepting.

Comment 30

[jglick]

These were in an archived spec:

http://gooey/client/5.0/specs/mail/accountSetup/archive/12_9_99/images/AcctServe
rPop1.gif
http://gooey/client/5.0/specs/mail/accountSetup/archive/12_9_99/images/AcctServe
rImap1.gif

But got removed for the final product. I'm assuming we'd want to do this the way 
originally spec'd?

Comment 31

[(not reading, please use [e-mail] instead)]

the ui and pref work has landed, now to make it actually log in at start up.

Comment 32

[(not reading, please use [e-mail] instead)]

ui is there, but it doesn't work yet.  updating summary.

Comment 33

[kmurray1115]

Seth, will a user fail over dialup trying to log in to AOL and an Earthlink 
account simultaneously, if both accounts are checked "Log in to this account at 
Startup?" If so, Jen, do we need to add UI that warns them of this condition?
Example: "If you are choosing to log in to multiple accounts on startup, some 
providers do not allow you to log in over a different provider's connection..." 
Am I missing something?

Comment 34

[(not reading, please use [e-mail] instead)]

my fix for this depends on racham's fix for #60668

Comment 35

[(not reading, please use [e-mail] instead)]

note, I'm switching to jglick's wording:  "Log onto this account at startup of
Mail"

kmurray: by default, we would log into both at startup.  (the default action is
to login at start up.)

if a user hit that problem, they'd have to disable "logon at startup".

how common is it for people to have multiple accounts that they can't access
like that?

Comment 36

[kmurray1115]

Not sure, but I'm willing to bet it's something we need QA to check into. nbaca, 
can you test the scenario I described above:
1. a user has AOL, and are connected via AOL, would like to use Seamonkey, but 
also have an Earthlink account - what happens if they have this pref set to log 
on to the Earthlink account, and
2. vice versa, if a user is dialed in via Earthlink, what happens if they have 
this pref set *on* for their AOL account?  

Comment 37

[(not reading, please use [e-mail] instead)]

racham has my fix, so he's going to run with this one.

I know racham and kmurray have some "issues-meeting" type questions for jglick.

Comment 38

[Håkan Waara]

*** Bug 66063 has been marked as a duplicate of this bug. ***

Comment 39

[racham]

Fix has been checked in.

Now user can turn 'on' this option (Login at startup) for all the accounts of
hos/her interest. We shall see all those servers getting messages at startup. If
required a password dialog will thrown.

Also, for all new accounts this option will be turned 'off' by default. User has
to manually turn it 'on' to include the account in Login at startup list.

Marking Fixed.

Comment 40

[Shawn]

I don't know if it's fixed or not.  I just installed Mozilla 2001012721 and it
checks the account at startup, regardless of whether the option is on or off.

Comment 41

[racham]

Please follow bug 66460 for default account and Login at startup issues.

Comment 42

[fenella]

Linux (2001-01-29-08 Mtrunk)
Win32 (2001-01-29-06 Mtrunk)
Mac (2001-01-29-09 Mtrunk)
This problem is fixed.