From 7a6d95f70ccb12d61355ecd677eff40dd13260b7 Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Fri, 14 Oct 2016 02:28:49 +0200 Subject: [PATCH] E-mail preferences page --- .../features/ui/components/navigation_bar.jsx | 2 +- app/assets/stylesheets/application.scss | 19 +++++++++++++ .../settings/preferences_controller.rb | 27 +++++++++++++++++++ .../profiles_controller.rb} | 4 +-- app/helpers/settings_helper.rb | 2 -- app/views/auth/registrations/edit.html.haml | 3 +++ app/views/settings/preferences/show.html.haml | 22 +++++++++++++++ .../settings/{ => profiles}/show.html.haml | 6 +++-- app/views/settings/shared/_links.html.haml | 7 +++++ config/routes.rb | 6 ++++- .../settings/preferences_controller_spec.rb | 16 +++++++++++ .../profiles_controller_spec.rb} | 2 +- spec/helpers/settings_helper_spec.rb | 5 ---- 13 files changed, 107 insertions(+), 14 deletions(-) create mode 100644 app/controllers/settings/preferences_controller.rb rename app/controllers/{settings_controller.rb => settings/profiles_controller.rb} (73%) delete mode 100644 app/helpers/settings_helper.rb create mode 100644 app/views/settings/preferences/show.html.haml rename app/views/settings/{ => profiles}/show.html.haml (64%) create mode 100644 app/views/settings/shared/_links.html.haml create mode 100644 spec/controllers/settings/preferences_controller_spec.rb rename spec/controllers/{settings_controller_spec.rb => settings/profiles_controller_spec.rb} (78%) delete mode 100644 spec/helpers/settings_helper_spec.rb diff --git a/app/assets/javascripts/components/features/ui/components/navigation_bar.jsx b/app/assets/javascripts/components/features/ui/components/navigation_bar.jsx index a168525411..68b660ee89 100644 --- a/app/assets/javascripts/components/features/ui/components/navigation_bar.jsx +++ b/app/assets/javascripts/components/features/ui/components/navigation_bar.jsx @@ -19,7 +19,7 @@ const NavigationBar = React.createClass({
{this.props.account.get('acct')} - Settings · Public timeline · Logout + Settings · Public timeline · Logout
); diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss index 60875a3b3f..ac21c809f2 100644 --- a/app/assets/stylesheets/application.scss +++ b/app/assets/stylesheets/application.scss @@ -214,6 +214,25 @@ body { } } + .fields-group { + margin-bottom: 25px; + } + + .boolean-field { + margin-bottom: 5px; + + label { + font-family: 'Roboto'; + font-size: 14px; + color: #9baec8; + } + + input[type=checkbox] { + display: inline-block; + margin-bottom: -13px; + } + } + input[type=text], input[type=email], input[type=password], textarea { background: transparent; border: 0; diff --git a/app/controllers/settings/preferences_controller.rb b/app/controllers/settings/preferences_controller.rb new file mode 100644 index 0000000000..8a038f2fe4 --- /dev/null +++ b/app/controllers/settings/preferences_controller.rb @@ -0,0 +1,27 @@ +class Settings::PreferencesController < ApplicationController + layout 'auth' + + before_action :authenticate_user! + + def show + end + + def update + current_user.settings(:notification_emails).follow = user_params[:notification_emails][:follow] == '1' + current_user.settings(:notification_emails).reblog = user_params[:notification_emails][:reblog] == '1' + current_user.settings(:notification_emails).favourite = user_params[:notification_emails][:favourite] == '1' + current_user.settings(:notification_emails).mention = user_params[:notification_emails][:mention] == '1' + + if current_user.save + redirect_to settings_preferences_path, notice: 'Changes successfully saved!' + else + render action: :show + end + end + + private + + def user_params + params.require(:user).permit(notification_emails: [:follow, :reblog, :favourite, :mention]) + end +end diff --git a/app/controllers/settings_controller.rb b/app/controllers/settings/profiles_controller.rb similarity index 73% rename from app/controllers/settings_controller.rb rename to app/controllers/settings/profiles_controller.rb index 299e1f3bc3..52b6369a6d 100644 --- a/app/controllers/settings_controller.rb +++ b/app/controllers/settings/profiles_controller.rb @@ -1,4 +1,4 @@ -class SettingsController < ApplicationController +class Settings::ProfilesController < ApplicationController layout 'auth' before_action :authenticate_user! @@ -9,7 +9,7 @@ class SettingsController < ApplicationController def update if @account.update(account_params) - redirect_to settings_path, notice: 'Changes successfully saved!' + redirect_to settings_profile_path, notice: 'Changes successfully saved!' else render action: :show end diff --git a/app/helpers/settings_helper.rb b/app/helpers/settings_helper.rb deleted file mode 100644 index ffbedbaf46..0000000000 --- a/app/helpers/settings_helper.rb +++ /dev/null @@ -1,2 +0,0 @@ -module SettingsHelper -end diff --git a/app/views/auth/registrations/edit.html.haml b/app/views/auth/registrations/edit.html.haml index 9a52af35c6..78f4ef5dc8 100644 --- a/app/views/auth/registrations/edit.html.haml +++ b/app/views/auth/registrations/edit.html.haml @@ -10,5 +10,8 @@ = f.password_field :password_confirmation, autocomplete: "off", placeholder: 'Confirm new password' .field = f.password_field :current_password, autocomplete: "off", placeholder: 'Current password' + .actions = f.button "Save changes", type: 'submit' + +.form-footer= render "settings/shared/links" diff --git a/app/views/settings/preferences/show.html.haml b/app/views/settings/preferences/show.html.haml new file mode 100644 index 0000000000..9c73d07c5b --- /dev/null +++ b/app/views/settings/preferences/show.html.haml @@ -0,0 +1,22 @@ +- content_for :page_title do + Preferences + += form_for current_user, url: settings_preferences_path, html: { method: :put } do |f| + = f.fields_for :notification_emails, current_user.settings(:notification_emails) do |ff| + .boolean-field + = ff.check_box :follow + = ff.label :follow, 'Send e-mail when someone follows you' + .boolean-field + = ff.check_box :reblog + = ff.label :reblog, 'Send e-mail when someone reblogs your status' + .boolean-field + = ff.check_box :favourite + = ff.label :favourite, 'Send e-mail when someone favourites your status' + .boolean-field + = ff.check_box :mention + = ff.label :mention, 'Send e-mail when someone mentions you' + + .actions + = f.button 'Save changes', type: :submit + +.form-footer= render "settings/shared/links" diff --git a/app/views/settings/show.html.haml b/app/views/settings/profiles/show.html.haml similarity index 64% rename from app/views/settings/show.html.haml rename to app/views/settings/profiles/show.html.haml index fe4d37b55c..2ff91beff1 100644 --- a/app/views/settings/show.html.haml +++ b/app/views/settings/profiles/show.html.haml @@ -1,7 +1,7 @@ - content_for :page_title do Edit profile -= form_for @account, url: settings_path, html: { method: :put } do |f| += form_for @account, url: settings_profile_path, html: { method: :put } do |f| .field = f.text_field :display_name, placeholder: 'Display name' .field @@ -14,4 +14,6 @@ = f.file_field :header .actions - = f.button 'Save changes', type: :submit + = f.button 'Save changes', type: :submit + +.form-footer= render "settings/shared/links" diff --git a/app/views/settings/shared/_links.html.haml b/app/views/settings/shared/_links.html.haml new file mode 100644 index 0000000000..520300416a --- /dev/null +++ b/app/views/settings/shared/_links.html.haml @@ -0,0 +1,7 @@ +%ul.no-list + - if controller_name != 'profiles' + %li= link_to "Edit profile", settings_profile_path + - if controller_name != 'preferences' + %li= link_to "Preferences", settings_preferences_path + - if controller_name != 'registrations' + %li= link_to "Change password", edit_user_registration_path diff --git a/config/routes.rb b/config/routes.rb index 94507d3248..ec0309dc7b 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -31,7 +31,11 @@ Rails.application.routes.draw do end end - resource :settings, only: [:show, :update] + namespace :settings do + resource :profile, only: [:show, :update] + resource :preferences, only: [:show, :update] + end + resources :media, only: [:show] namespace :api do diff --git a/spec/controllers/settings/preferences_controller_spec.rb b/spec/controllers/settings/preferences_controller_spec.rb new file mode 100644 index 0000000000..a5d349d6df --- /dev/null +++ b/spec/controllers/settings/preferences_controller_spec.rb @@ -0,0 +1,16 @@ +require 'rails_helper' + +RSpec.describe Settings::PreferencesController, type: :controller do + + before do + sign_in Fabricate(:user), scope: :user + end + + describe "GET #show" do + it "returns http success" do + get :show + expect(response).to have_http_status(:success) + end + end + +end diff --git a/spec/controllers/settings_controller_spec.rb b/spec/controllers/settings/profiles_controller_spec.rb similarity index 78% rename from spec/controllers/settings_controller_spec.rb rename to spec/controllers/settings/profiles_controller_spec.rb index 0b41a2111c..526bbc5bce 100644 --- a/spec/controllers/settings_controller_spec.rb +++ b/spec/controllers/settings/profiles_controller_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe SettingsController, type: :controller do +RSpec.describe Settings::ProfilesController, type: :controller do before do sign_in Fabricate(:user), scope: :user diff --git a/spec/helpers/settings_helper_spec.rb b/spec/helpers/settings_helper_spec.rb deleted file mode 100644 index 9c68490a1c..0000000000 --- a/spec/helpers/settings_helper_spec.rb +++ /dev/null @@ -1,5 +0,0 @@ -require 'rails_helper' - -RSpec.describe SettingsHelper, type: :helper do - -end