add title to profile notifications page

fixes VICE-1780
flag=none

test plan:
  - navigate to /profile/communication
    - the page title should be 'Account Notification Settings'

qa risk: low

Change-Id: Iea87b401973147e98b1c760602b343d7ce1f68d0
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/270395
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Drake Harper <drake.harper@instructure.com>
QA-Review: Drake Harper <drake.harper@instructure.com>
Product-Review: Davis Hyer <dhyer@instructure.com>
This commit is contained in:
Davis Hyer 2021-07-30 11:09:25 -06:00
parent 860e25acac
commit a29a98dec2
2 changed files with 13 additions and 0 deletions

View File

@ -234,6 +234,7 @@ class ProfileController < ApplicationController
@user = @current_user
@current_user.used_feature(:cc_prefs)
@context = @user.profile
@page_title = t('account_notification_settings_title', 'Account Notification Settings')
set_active_tab 'notifications'
add_crumb(@current_user.short_name, profile_path)

View File

@ -331,4 +331,16 @@ describe ProfileController do
end
end
end
describe "communication" do
context "when rendering the full view" do
render_views
it "sets the appropriate page title" do
user_session(@user)
get "communication"
expect(response.body).to include "<title>Account Notification Settings</title>"
end
end
end
end