-
Notifications
You must be signed in to change notification settings - Fork 47
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
Extend 'Client' class with 'create_multiple' and 'bulk_delete' methods #38
base: master
Are you sure you want to change the base?
Conversation
…oad (extended 'MessageBuilder' with 'create_multiple_request' and created 'ExecuteMultipleResult' class);
…lete’ method (accepting ‘criteria’ array);
…y forcing bundler gem update (ref: travis-ci/travis-ci#3531);
…t for the finish of bulk job execution (‘check_job_status’ helper method introduced);
…ation Improve region determination
@@ -7,3 +7,5 @@ notifications: | |||
- [email protected] | |||
on_success: never | |||
on_failure: always | |||
before_install: | |||
- gem install bundler |
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.
Why was this necessary?
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.
Build on Travis CI did not succeed, due to:
"In short, the bundler version that Travis uses is very outdated. Running a gem update and install for bundler as a prestep for the Travis build fixes it usually."
Please refer to the issues described here:
@karolkulesza Thank you for the contribution. I've posted a couple comments. |
@karolkulesza Thanks for the information. Could you provide some level of spec coverage to go with this change? |
…arties on create This allows a user to create entities like: from1 = DynamicsCRM::XML::Entity.new('activityparty') from1.attributes = DynamicsCRM::XML::Attributes.new({partyid: DynamicsCRM::XML::EntityReference.new("systemuser", @employee.id)}) to1 = DynamicsCRM::XML::Entity.new('activityparty') to1.attributes = DynamicsCRM::XML::Attributes.new({partyid: DynamicsCRM::XML::EntityReference.new("lead", @lead.id) }) body = { subject: "Subject for your phonecall", description: "Description", from: [from1], to: [to1] } @call = @client.create('phonecall', body)
Support ArrayOfEntity so one can create bound entities like ActivityParties on create
@karolkulesza Could you rebase this branch? I think it will fix the build issue. Also, I'd like to see a spec added for this. |
Add special case for nil value
…ttributes in DynamicsCRM::Client;
@karolkulesza Could you rebase this branch? I think it will fix the build issue. |
…Fault for 'ExecuteMultipleResponseItem');
…oad (extended 'MessageBuilder' with 'create_multiple_request' and created 'ExecuteMultipleResult' class);
…lete’ method (accepting ‘criteria’ array);
…y forcing bundler gem update (ref: travis-ci/travis-ci#3531);
…t for the finish of bulk job execution (‘check_job_status’ helper method introduced);
…ttributes in DynamicsCRM::Client;
…Fault for 'ExecuteMultipleResponseItem');
Fixed my build issues. Seems like the only one left is related with 'On Premise authentication':
|
…oad (extended 'MessageBuilder' with 'update_multiple_request');
…ete (extended 'MessageBuilder' with 'delete_multiple_request');
…Target'" in case of multiple delete request;
@karolkulesza I'd like to see this work get contributed, but your branch is quite diverged at this point. Can you rebrase this branch from the latest master so we can see the actual differences? |
@karolkulesza Would you be able to re-base your branch? I'd like to get this feature added. |
No description provided.