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

Added configurable options to the gem #37

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ivobenedito
Copy link
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
Copy link
Owner

fredwu commented Jan 28, 2013

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

@jimhj
Copy link

jimhj commented Jan 31, 2013

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

@ivobenedito
Copy link
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
Copy link

jsallis commented Feb 20, 2013

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
Copy link
Owner

fredwu commented Feb 20, 2013

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

@fredwu
Copy link
Owner

fredwu commented Jul 19, 2013

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