-
Notifications
You must be signed in to change notification settings - Fork 6
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
SFSDK-7 Adds a delete and codebase endpoints #8
Conversation
lib/sfrest/codebase.rb
Outdated
@@ -0,0 +1,16 @@ | |||
module SFRest | |||
# List codebases on the |
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.
incomplete sentence
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
lib/sfrest/codebase.rb
Outdated
end | ||
|
||
# Lists the codebases | ||
# @return [Hash] A hash of codebases configured for the factory |
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.
sentence ending period
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
lib/sfrest/site.rb
Outdated
@@ -83,13 +83,28 @@ def site_list | |||
# @param [String] sitename The name of the site to create | |||
# @param [Integer] group_id The Id of the group the site is to be a member of | |||
# @param [String] install_profile The install profile to use when creating the site | |||
def create_site(sitename, group_id, install_profile = nil) | |||
def create_site(sitename, group_id, install_profile = nil, codebase = nil) |
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.
missing param section for codebase
lib/sfrest/site.rb
Outdated
# @param [String] sitename The name of the site to create | ||
# @param [Integer] group_id The Id of the group the site is to be a member of | ||
# @param [String] install_profile The install profile to use when creating the site | ||
alias create create_site |
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.
missing param section for codebase
spec/site_spec.rb
Outdated
site_id = site_data['site_id'] | ||
site_name = site_data['site'] | ||
task_id = site_data['task_id'] | ||
it 'can create a site' do |
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.
it 'can delete a site' do
- add params for codebase in create - fix incorrect name in site delete spec - missing punctuation
No description provided.