Improvements and fixes on OAuth providers (#304)

.gitignore

@@ -10,6 +10,9 @@
 # Ignore vs code config.
 .vscode
 
+# Ignore all .DS_Store files.
+**/.DS_Store
+
 # Ignore bundler config.
 /.bundle
 

app/controllers/o_auths_controller.rb

@@ -2,6 +2,7 @@ class OAuthsController < ApplicationController
   include HTTParty
   include OAuthsHelper
   include ApplicationHelper
+  include Devise::Controllers::Rememberable
 
   before_action :authenticate_admin, only: [:index, :create, :update, :destroy]
 
@@ -124,6 +125,7 @@ class OAuthsController < ApplicationController
 
     if user.oauth_token == params[:token]
       sign_in user
+      remember_me user
       user.invalidate_oauth_token
       flash[:notice] = I18n.t('devise.sessions.signed_in')
       redirect_to root_path

app/controllers/tenants_controller.rb

@@ -46,9 +46,6 @@ class TenantsController < ApplicationController
       @user.save!
 
       CreateWelcomeEntitiesWorkflow.new().run
-      OAuth.include_only_defaults.each do |o_auth|
-        TenantDefaultOAuth.create(o_auth_id: o_auth.id)
-      end
 
       logger.info { "New tenant registration: #{Current.tenant.inspect}" }
 

app/javascript/components/SiteSettings/Authentication/OAuthProviderItem.tsx

@@ -28,7 +28,12 @@ const OAuthProviderItem = ({
 }: Props) => (
   <li className="oAuthListItem">
     <div className="oAuthInfo">
+      {
+        oAuth.logo && oAuth.logo.length > 0 ?
           <img src={oAuth.logo} className="oAuthLogo" width={42} height={42} />
+        :
+          <div className="oauthLogo" style={{width: 42, height: 42}}></div>
+      }
 
       <div className="oAuthNameAndEnabled">
         <span className="oAuthName">{oAuth.name}</span>

app/javascript/components/TenantSignUp/TenantSignUpP.tsx

@@ -91,7 +91,7 @@ const TenantSignUpP = ({
       if (authMethod == 'oauth') {
         let redirectUrl = new URL(baseUrl);
         redirectUrl.hostname = `${subdomain}.${redirectUrl.hostname}`;
-        window.location.href = `${redirectUrl.toString()}users/sign_in`;
+        window.location.href = redirectUrl.toString();
         return;
       }
 

app/javascript/components/common/OAuthProviderLink.tsx

@@ -14,7 +14,7 @@ const OAuthProviderLink = ({ oAuthId, oAuthName, oAuthLogo, oAuthReason, isSignU
     onClick={() => window.location.href = `/o_auths/${oAuthId}/start?reason=${oAuthReason}`}
     className={`oauthProviderBtn oauthProvider${oAuthName.replace(' ', '')}`}
   >
-    <img src={oAuthLogo} alt={oAuthName} width={28} height={28} />
+    { oAuthLogo && oAuthLogo.length > 0 && <img src={oAuthLogo} alt={oAuthName} width={28} height={28} /> }
     <span className='oauthProviderText'>
       {
         isSignUp ?

app/workflows/create_welcome_entities_workflow.rb

@@ -67,5 +67,13 @@ class CreateWelcomeEntitiesWorkflow
       body: 'Users can comment to express their opinions! As with posts and board descriptions, comments can be *Markdown* **formatted**',
       user_id: owner.id
     )
+
+    # Set first board as root page
+    TenantSetting.create!(root_board_id: feature_board.id)
+
+    # Enable all default oauths
+    OAuth.include_only_defaults.each do |o_auth|
+      TenantDefaultOAuth.create!(o_auth_id: o_auth.id)
+    end
   end
 end