Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #920 (closed defect: wontfix)

Opened 11 years ago

Last modified 10 years ago

change identity prompt in quickstart

Reported by: fredlin Owned by: anonymous
Priority: low Milestone: 0.9
Component: Identity Version: 0.9a6
Severity: trivial Keywords: quickstart
Cc:

Description

origin identity prompt in quickstart is:

Do you need Identity (usernames/passwords) in this project? [no]:

The identity prompt in quickstart notice me to type 'yes' to trigger identity functions,

and it's annoying to type 'yes', 'no' to trigger Identity template(I've to use two hands to type that:P ).

I just found to accept abbrev type (y/n) is possible in the code (quickstart.py),

So why not change '[no]' to unix type 'yes/no[default value]' for better guidance(cause its a 'quickstart')

The prompt is:

Do you need Identity (usernames/passwords) in this project? yes/no[n]

and for consist, I suggest to change following prompt from

Select your provider sqlobject or sqlalchemy [sqlobject]:

to

Select your provider: sqlobject/sqlalchemy[sqlobject]:

and make 'so' or 'sa' can be recognize as 'sqlobject' or 'sqlalchemy'

code is here(in quickstart.py)

  • quickstart.py

     
    131131        doidentity = None 
    132132        while not doidentity: 
    133133            doidentity = raw_input("Do you need Identity " 
    134                         "(usernames/passwords) in this project? [no] ") 
     134                        "(usernames/passwords) in this project? yes/no[n]: ") 
    135135            doidentity = doidentity.lower() 
    136136            if not doidentity or doidentity.startswith('n'):  
    137137                self.identity="none" 
     
    142142            doidentity = None 
    143143 
    144144        while not self.identity: 
    145             self.identity = raw_input("Select your provider sqlobject or sqlalchemy [sqlobject]: ") 
     145            self.identity = raw_input("Select your provider: sqlobject/sqlalchemy [sqlobject]: ") 
     146            if (self.identity=='so'): 
     147                self.identity== "sqlobject" 
     148            if (self.identity=='sa'): 
     149                self.identity== "sqlalchemy" 
    146150            if not self.identity: 
    147151                self.identity = "sqlobject" 
    148             if self.identity not in ["sqlobject", "sqlalchemy"]: 
     152            if self.identity not in ["sqlobject", "sqlalchemy", 'so', 'sa']: 
    149153                print "Please enter one of 'sqlobject(so)', 'sqlalchemy(sa)'" 
    150154                self.identity = "" 
    151155                continue 

Change History

comment:1 Changed 11 years ago by jorge.vargas

not to be pesimist but

the current prompt makes it easy for people to find out that y is posible, and come on the questions has 2 possible anwsers so the yes is implicit.

On the other hand all reference to this should be look and change in the wiki/docs

about the sqlobject part your current code will fail when sqlalchemy is type in which is what the raw_input suggest.

On the other hand think of it this way. The idea is press enter if your going with defaults, and your done and that's the main goal of quickstart.

On the other hand remenber TG is not a set of interchangable components is the best one for each part with enough flexibility for not becomming a pain to go around the framework.

comment:2 Changed 11 years ago by kevin

  • Status changed from new to closed
  • Resolution set to wontfix

I like the look of the existing one, and the second prompt is going away (already gone in my local copy). So, I'm going to close this ticket.

Note: See TracTickets for help on using tickets.