On Mon, Jul 17, 2017 at 3:18 PM, Craig Russell <[email protected]> wrote: > I've checked in the proposed icla-invalid-id message. I cannot figure out how > to activate it. > > There still needs to be a check in icla.js.rb for invalid id. But if it is > illegal, the (File) button should be enabled. This allows icla.json.rb to > construct the appropriate email. > > Perhaps > > if @user and not @checked > reply = 'icla-invalid-id.erb' > else if ... > > But then later, instead of > if @user and not @user.empty? > > add a check for @checked > if @user and @checked and not @user.empty? > > Seems easy enough but I'm not quite good enough with js.rb.json stuff.
If I'm understanding correctly: On the client side, you no longer want the File button to be disabled if the provided user id is invalid. Furthermore, if the user id is invalid, the validity of the project and votelink fields should no longer affect whether the File button is enabled or not. On the server side, the user id provided needs to be checked, and if it doesn't match the regular expression or isn't available, then a different reply template should be used and the new account request should be skipped. The following change should do the above: https://github.com/apache/whimsy/commit/6d6409756c67a95216ecf670ea7baded064bfcf4 Note: I made a small tweak to the invalid-id-request reply to clarify that the id must be lowercase. > Craig - Sam Ruby >> On Jul 17, 2017, at 11:01 AM, Craig Russell <[email protected]> wrote: >> >> Currently the secmail tool properly checks a proposed id and highlights it >> in red if it does not conform. Either it is already in use or it is not >> alphabetic followed by at least two alphanumeric characters. If the id is >> illegal, the tool refuses to file the icla. Then everyone complains that the >> account was not created. >> >> It comes up often enough that I'd like a different way of handling this case. >> >> If the name is illegal, I'd like to file the icla but send a different >> message. >> >> I'll work on the message and then work on the processing to send it. >> >> Craig >> >> >> Craig L Russell >> Secretary, Apache Software Foundation >> [email protected] <mailto:[email protected]> http://db.apache.org/jdo >> <http://db.apache.org/jdo> > > Craig L Russell > Secretary, Apache Software Foundation > [email protected] <mailto:[email protected]> http://db.apache.org/jdo > <http://db.apache.org/jdo>
