Skip to content

Added configurable options to the gem#37

Open
ivobenedito wants to merge 5 commits into
fredwu:masterfrom
ivobenedito:added-configurable
Open

Added configurable options to the gem#37
ivobenedito wants to merge 5 commits into
fredwu:masterfrom
ivobenedito:added-configurable

Conversation

@ivobenedito

Copy link
Copy Markdown
Contributor

Hi,

I've added configurable options to the gem so that you can customize some parameters.

Example:

ApiTaster.configure do |config|
  config.title = 'MyTitle'
  config.call_to_action = 'My call to action text!'
  config.app_title = 'MyAppTitle'
  config.app_url = 'http://www.myapp.com'
  config.include_missing_definitions = false
  config.include_obsolete_definitions = false
end

By default, the gem is using these values:

config.title = 'API Taster'
config.call_to_action = 'Select an API endpoint on the left to get started. :)'
config.app_title = 'Application'
config.app_url = '/'
config.include_missing_definitions = true
config.include_obsolete_definitions = true

Feel free to merge it if you want.

@fredwu

fredwu commented Jan 28, 2013

Copy link
Copy Markdown
Owner

Very nice, thanks for this! :) Before I merge this in though would you add some tests please?

@jimhj

jimhj commented Jan 31, 2013

Copy link
Copy Markdown

I need to use api_tester in multi applications, this merge will be very helpful to me.

@ivobenedito

Copy link
Copy Markdown
Contributor Author

Very nice, thanks for this! :) Before I merge this in though would you add some tests please?

Sure, I will.
Just don't have much free time at the moment, but as soon as I can I'll add them.

@jsallis

jsallis commented Feb 20, 2013

Copy link
Copy Markdown

Any update on this pull request? I'm running into a problem that this would solve and I'd rather use the official source than patching my own version if possible.

@fredwu

fredwu commented Feb 20, 2013

Copy link
Copy Markdown
Owner

As soon as tests are added I'll merge this in. :) cc @ivobenedito

@fredwu

fredwu commented Jul 19, 2013

Copy link
Copy Markdown
Owner

Hi @ivobenedito do you reckon you'd have time to write some tests for this? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants