diff --git a/web/app/assets/stylesheets/web/main.css.scss b/web/app/assets/stylesheets/web/main.css.scss index c680de296..6abeebc01 100644 --- a/web/app/assets/stylesheets/web/main.css.scss +++ b/web/app/assets/stylesheets/web/main.css.scss @@ -106,6 +106,10 @@ body.web { margin-right:0px; } + input.button-orange { + height:26px; + } + .button-grey:hover { cursor:default; background-color:#666; @@ -290,10 +294,6 @@ body.web { .actions { margin-top:20px; - - a.button-grey { - line-height:15px; // WHY is this not universal - } } .error { @@ -332,9 +332,6 @@ body.web { } - - - .right-side { margin-left:25px; } diff --git a/web/app/controllers/users_controller.rb b/web/app/controllers/users_controller.rb index c80d709e2..1420b2012 100644 --- a/web/app/controllers/users_controller.rb +++ b/web/app/controllers/users_controller.rb @@ -232,15 +232,16 @@ class UsersController < ApplicationController UserMailer.welcome_message(@user).deliver sign_in @user redirect_to :client + return elsif !@user.nil? # new user with validation errors; logger.debug("#{@user} has errors. can not sign in until remedied. #{@user.errors.inspect}") - end # let page have signup_token in javascript gon.signup_token = signup_token # let errors fall through to signup_confirm.html.erb + render :layout => 'web' end def edit diff --git a/web/app/views/users/signup_confirm.html.erb b/web/app/views/users/signup_confirm.html.erb index ead385897..0070ca9dd 100644 --- a/web/app/views/users/signup_confirm.html.erb +++ b/web/app/views/users/signup_confirm.html.erb @@ -2,116 +2,17 @@ <%= javascript_include_tag "landing/landing" %> <% if @user.nil? %> -

Signup Confirmation Failure

+

Signup Already Confirmed

-
-
- Unable to confirm registration email -
-
+
+
+

+ If you are seeing this page, you have probably already confirmed this email address. +

+
+

+ Please proceed to <%= link_to 'the home page', root_path %> and log in. +

<% else %> - <% # Seth: I'm commenting out all non-required fields for now, because I think we currently have too much %> - -
- - -
-

create a jamkazam account

-
- - -
- <%= form_for(@user) do |f| %> - - -
-
- - - - - - - - - - - - - - - - - - - - - - - -
<%= f.label :first_name, "First Name" %>
- <%= f.text_field :first_name %>
<%= f.label :last_name, "Last Name" %>
- <%= f.text_field :last_name %>
<%= f.label :email %>
- <%= f.text_field :email, :disabled => true %>
Country:
- -
State/Province:
-
City:
-
<%= f.label :password %>
- <%= f.password_field :password %>
<%= f.label :password_confirmation, "Verify Password" %>
- <%= f.password_field :password_confirmation %>
I will accept email from JamKazam about this service.
-
- I have read and agree to the JamKazam <%= link_to "terms of service", corp_terms_path , :rel=>"external" %>.
- -
- - -
- - -

- - What instruments can you play? - -
- <%= render :partial => "instrument_selector"%> -
- -
- <% if Rails.application.config.recaptcha_enable %> - <%= recaptcha_tags %> - <% end %> - - - - <%= f.submit "CREATE ACCOUNT", class: "right button-orange", id: "completion_submit_button" %>

- - -
- -
- - - <% end %> -
- - - - - <% end %> - - + The server had a problem. Please try to confirm your email later. +<% end %> diff --git a/web/config/environments/production.rb b/web/config/environments/production.rb index 53083251b..3685fe7dc 100644 --- a/web/config/environments/production.rb +++ b/web/config/environments/production.rb @@ -9,7 +9,7 @@ SampleApp::Application.configure do config.action_controller.perform_caching = true # Disable Rails's static asset server (Apache or nginx will already do this) - config.serve_static_assets = false + config.serve_static_assets = true # in true production we use false, but in 'fake production mode locally', this is useful # Compress JavaScripts and CSS config.assets.compress = false # this seems like a bad idea in early development phases