Merge from intercom origin#1
Open
davidsickmiller wants to merge 49 commits into
Open
Conversation
Migrate to Circle2.0
This `$json` parameter passed as a Guzzle `json` request option is intended to be an `array` as referenced in the calling Intercom endpoint classes.
This change introduces a new `archiveUser` method, and updates the `deleteUser` method to alias it. Additionally the change introduces a new `permanentlyDeleteUser` that leverages a `POST` to `user_delete_requests` to initiate a permanent delete from the intercom api. Addresses: intercom#249
Adding PR template
Fix license identifier
…type-for-json Update docblock type for `$json` parameter
Get a single company by ID
Fixes to README
* Fix readme examples styling * Fix code styling issues
* Adding info on delete and archive updates in README * Fix comment block
* Add Intercom\Test namespace to all the tests * Enforce PSR2 by CI * Fix PHPUnit imports * Import PHPUnit_Framework_TestCase
Co-Authored-By: murtron <[email protected]>
Add event metadata example to README
* Upgrade to PHP 7 * Update README.md Co-Authored-By: GabrielAnca <[email protected]>
* Add http-plug and remove guzzle * Move client initialization to the constructor * Allow HTTP Plug 1.0 or 2.0 * Update tests * Clarify readme * Update error handling section in README * Rename setClient to setHttpClient * Minor improvements * Fix exception handling * Change client implementation link in readme
Signed-off-by: Julien Bonvarlet <[email protected]>
Co-Authored-By: Gabriel Anca Corral <[email protected]>
Co-Authored-By: Gabriel Anca Corral <[email protected]>
Add customer search
…ercom#287) * Add support for listing users of a company * Remove unneeded method and add to readme * Remove unneeded test
Adding missing tag above visitors section so it's not inside the codeblock.
Co-authored-by: Gabriel Anca Corral <[email protected]>
* Abstract to IntercomResources. * Fix comment. Co-authored-by: Gabriel Anca Corral <[email protected]>
Co-authored-by: Gabriel Anca Corral <[email protected]>
* Prefer assertSame instead of assertEquals. * Use assertIsArray. * Use assertSame instead of assertTrue. * Fix expected and actual. * Create the client mock in an abstract TestCase. * Update test/TestCase.php Co-Authored-By: Gabriel Anca Corral <[email protected]> * Update test/IntercomClientTest.php Co-Authored-By: Gabriel Anca Corral <[email protected]> Co-authored-by: Gabriel Anca Corral <[email protected]>
…o merge-from-intercom-origin
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I believe this will bring us up to date with the last two years of changes from intercom/intercom-php.