Skip to content

Commit

Permalink
Merge pull request #216 from zendesk/asilva/remove-role-option-theming
Browse files Browse the repository at this point in the history
Remove role option from Theming Center tool
  • Loading branch information
augustocravosilva authored May 6, 2019
2 parents cae69d7 + 8dec9f7 commit ed9a189
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions lib/zendesk_apps_tools/theme.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,6 @@ class Theme < Thor
shared_options(except: %i[clean unattended])
method_option :port, default: Command::DEFAULT_SERVER_PORT, required: false, desc: 'Port for the http server to use.'
method_option :bind, required: false
method_option :role,
type: :string,
enum: %w[manager agent end_user anonymous],
default: 'manager',
desc: 'The role for the preview URL',
aliases: '-r'
method_option :livereload, type: :boolean, default: true, desc: 'Enable or disable live-reloading the preview when a change is made.'
def preview
setup_path(options[:path])
Expand All @@ -32,7 +26,7 @@ def preview
no_commands do
def full_upload
say_status 'Generating', 'Generating theme from local files'
payload = generate_payload.merge(role: options[:role])
payload = generate_payload
say_status 'Generating', 'OK'
say_status 'Uploading', 'Uploading theme'
connection = get_connection(nil)
Expand Down

0 comments on commit ed9a189

Please sign in to comment.