Commit 0f6964a3 authored by Sean McGivern's avatar Sean McGivern

Merge branch '25294-remove-signed-out-msg' into 'master'

fix: 25294 - remove signed_out notification

Closes #25294

See merge request !7958
parents db9e1635 3c36d9dc
...@@ -37,6 +37,12 @@ class SessionsController < Devise::SessionsController ...@@ -37,6 +37,12 @@ class SessionsController < Devise::SessionsController
end end
end end
def destroy
super
# hide the signed_out notice
flash[:notice] = nil
end
private private
# Handle an "initial setup" state, where there's only one user, it's an admin, # Handle an "initial setup" state, where there's only one user, it's an admin,
......
---
title: 'fix: removed signed_out notification'
merge_request: 7958
author: jnoortheen
...@@ -75,7 +75,8 @@ module LoginHelpers ...@@ -75,7 +75,8 @@ module LoginHelpers
def logout def logout
find(".header-user-dropdown-toggle").click find(".header-user-dropdown-toggle").click
click_link "Sign out" click_link "Sign out"
expect(page).to have_content('Signed out successfully') # check the sign_in button
expect(page).to have_button('Sign in')
end end
# Logout without JavaScript driver # Logout without JavaScript driver
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment