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

Remove security workflow #109

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

Conversation

swati31196
Copy link

No description provided.

Abhay Rana and others added 30 commits July 29, 2015 01:08
    - refactor client code for posting messages through a guzzlePoster function
    - added retry mechanism to regular Message::send function
    - added better recovery mechanism to the Client queue class
… or via a queue

- implementing retry mechanism for handling queuing retries
- removing unnecessary variables
- fix job getting repushed post delete on failure
- method documentation changes
    - moving preparing payload to the message class
    - adding a seperate payload property with a getter for getting the payload
    - adding missing function flatten_array to un-flatten multi dimensional arrays into a singular one
    - minor refactoring to remove duplicated code
- numretries needs to be set differently for different payloads
- fix laravel 5 service provider class register function to pass in correct arguments to client class
- remove test code
gaurav and others added 30 commits March 11, 2017 10:17
removed unused code fom phpunit xml conf
This causes inheritance issues when extending the class and can break a
queue job. Using `static::class` will ensure that the child class name
is always using for the job payload.
Don't use __CLASS__ to push job into the queue
[travis] Trusty for Travis
* Update Client.php

* Update Client.php

* Update ClientFunctionalTest.php

* Update ClientFunctionalTest.php

* Update Client.php

* Bring inline with maknz/slack

* Add icon into test for code coverage
Removes requirement for action confirmation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.