Skip to content
This repository has been archived by the owner on Sep 21, 2021. It is now read-only.

WIP Bugfix agent MQTT #661

Open
wants to merge 17 commits into
base: develop
Choose a base branch
from
Open

WIP Bugfix agent MQTT #661

wants to merge 17 commits into from

Conversation

ajsb85
Copy link
Contributor

@ajsb85 ajsb85 commented Dec 18, 2018

Changes description

Hotfixes by @rafaelje and @farfou

Checklist

Please check if your PR fulfills the following specifications:

  • Tests for the changes have been added
  • Docs have been added/updated

Estimated time

Assignee 🍅
@farfou 1
@rafaelje 1
@ajsb85 1

References

Closes #N/A
Related https://github.com/flyve-mdm/android-mdm-agent/issues/610
Depends on #N/A

rafaelje and others added 17 commits October 16, 2018 00:02
Signed-off-by: Rafa Hernandez <[email protected]>
Signed-off-by: Rafa Hernandez <[email protected]>
…enter) instances, along with a disconnect force call if MqttAndroidClient already exists
@ajsb85 ajsb85 added the bug label Dec 18, 2018
@ajsb85 ajsb85 added this to the 2.0 milestone Dec 18, 2018
@ajsb85 ajsb85 requested a review from teclib-bot December 18, 2018 01:43
@ghost
Copy link

ghost commented Dec 18, 2018

There were the following issues with this Pull Request

  • Commit: 12b2cd9
    • ✖ message may not be empty
    • ✖ type may not be empty
  • Commit: ca3cf20
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

@ajsb85 ajsb85 changed the title Bugfix agent MQTT WIP Bugfix agent MQTT Dec 18, 2018
Copy link
Contributor Author

@ajsb85 ajsb85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @rafaelje
Do you have plan to continue this work or can I delete the branch and close the PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants