-
-
Notifications
You must be signed in to change notification settings - Fork 359
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
AddPlaygroundAccess [finished, ready for merging] #1134
AddPlaygroundAccess [finished, ready for merging] #1134
Conversation
changes.add("access", "private"); | ||
} | ||
} | ||
@Override public String getCommitMessage() { return "Add whatever playground is public"; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not this be "Add whether playground is public"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I simplified the text here
app/src/main/res/values/strings.xml
Outdated
@@ -609,5 +609,6 @@ Otherwise, you can download another keyboard in the app store. Popular keyboards | |||
<string name="quest_address_answer_no_housenumber_message2">Is this correct and is it not just part of a building?</string> | |||
|
|||
<string name="notification_channel_download">"Download"</string> | |||
<string name="quest_playground_access_title">Is this playground generally accessible?</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*publicly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
The drawables are there now, btw |
7728ace
to
795cd49
Compare
795cd49
to
c243fe8
Compare
Thanks for the icon! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
fixes #1114