Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

Bug 815693: Rename .user-state #3310

Merged
merged 1 commit into from Jul 6, 2015
Merged

Bug 815693: Rename .user-state #3310

merged 1 commit into from Jul 6, 2015

Conversation

stephaniehobson
Copy link
Contributor

Renamed user-state to oauth-logged-in to mirror oauth-login class as the two components swap places depending on user state. (I know this seems picky but it's to support some other more complex changes).

Testing:

  • sign in, make sure signout widget displays correctly

Renamed user-state to oauth-logged-in to mirror oauth-login class as the two components swap places depending on user state. (I know this seems picky but it's to support some other more complex changes).
@openjck openjck self-assigned this Jul 6, 2015
@openjck
Copy link
Contributor

openjck commented Jul 6, 2015

Nice! This looks like all instances of user-state alright. Spot-checking now...

openjck added a commit that referenced this pull request Jul 6, 2015
@openjck openjck merged commit 1ae6b38 into mdn:master Jul 6, 2015
@stephaniehobson stephaniehobson deleted the bug-815693-userstate-rename branch July 13, 2015 22:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants