Skip to content

Commit

Permalink
Sharing: Remove the CLI share option
Browse files Browse the repository at this point in the history
Due to COPPA restrictions, sharing is not supported in Make Snake so
remove the option from the CLI parser. It has been removed in a
non-destructive way so should we choose to re-enable it then it simply
requires adding the button back.
  • Loading branch information
tombettany committed Jul 2, 2019
1 parent 00d8406 commit 1050a68
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 8 deletions.
6 changes: 6 additions & 0 deletions debian/changelog
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
make-snake (4.3.0-0) unstable; urgency=low

* Remove option to share from the python snake command

-- Team Kano <dev@kano.me> Tue, 2 Jul 2019 15:17:00 +0100

make-snake (3.14.0-0) unstable; urgency=low

* Applied changes from v3.9.2 es_AR backports to latest
Expand Down
18 changes: 10 additions & 8 deletions snake/parser.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
#!/usr/bin/env python

#
# parser.py
#
# Copyright (C) 2013, 2014 Kano Computing Ltd.
# License: http://www.gnu.org/licenses/gpl-2.0.txt GNU General Public License v2
# Copyright (C) 2013-2019 Kano Computing Ltd.
# License: http://www.gnu.org/licenses/gpl-2.0.txt GNU GPL v2
#
# Contributors: https://github.com/alexaverill
#
Expand Down Expand Up @@ -85,15 +85,17 @@ def init():
action="store_true", dest="reset", default=False,
help=_("Resets the game to challenge 1"))

parser.add_argument("--share",
action="store_true", dest="share", default=False,
help=_("Share your favourite theme with the world"))
# Disable sharing:
# parser.add_argument("--share",
# action="store_true", dest="share", default=False,
# help=_("Share your favourite theme with the world"))

args = parser.parse_args()

# Disable sharing:
# Check for share argument
if (args.share):
utils.share_theme()
# if (args.share):
# utils.share_theme()

# Check for valid theme
if (args.theme and args.theme != 'minimal'):
Expand Down

0 comments on commit 1050a68

Please sign in to comment.