Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

show correct dialog header text for master lock code. fixes #15543 #5720

Merged
merged 1 commit into from
Nov 16, 2014

Conversation

MartijnKaijser
Copy link
Member

old header: "Ask for master lock code on startup"
new header: "Master lock"

note:
is that comment that you see in the diff correct? because the dialog actually shows:

"Master code is not valid"
"Please enter a valid master code"

@MartijnKaijser MartijnKaijser added this to the Helix 14.0-beta3 milestone Nov 13, 2014
@MartijnKaijser MartijnKaijser added Type: Fix non-breaking change which fixes an issue Helix labels Nov 13, 2014
@tamland
Copy link
Member

tamland commented Nov 16, 2014

looks good to me. comment is obviously wrong, just remove it:)

MartijnKaijser added a commit that referenced this pull request Nov 16, 2014
show correct dialog header text for master lock code. fixes #15543
@MartijnKaijser MartijnKaijser merged commit c3ffcf9 into xbmc:master Nov 16, 2014
@MartijnKaijser
Copy link
Member Author

after having some beers the comment is actually correct. if you change mode the password is reset and you need to enter a new one. as such i adjusted the comment

@MartijnKaijser MartijnKaijser deleted the 15543 branch February 21, 2015 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Fix non-breaking change which fixes an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants