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

refactor(rn): start daemon #642

Merged
merged 18 commits into from
Nov 27, 2018
Merged

refactor(rn): start daemon #642

merged 18 commits into from
Nov 27, 2018

Conversation

90dy
Copy link
Member

@90dy 90dy commented Nov 22, 2018

fix #625
fix #624

@90dy 90dy requested a review from a team as a code owner November 22, 2018 09:29
@90dy 90dy self-assigned this Nov 22, 2018
@90dy 90dy added this to the w18.47 🌳 milestone Nov 22, 2018
@codecov
Copy link

codecov bot commented Nov 22, 2018

Codecov Report

Merging #642 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #642   +/-   ##
=======================================
  Coverage   22.24%   22.24%           
=======================================
  Files          50       50           
  Lines        2805     2805           
=======================================
  Hits          624      624           
  Misses       2034     2034           
  Partials      147      147
Impacted Files Coverage Δ
core/node/p2pclient.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7af249b...7a9a055. Read the comment docs.

@alexsland alexsland modified the milestones: w18.47 🌳, w18.48 🌧️ Nov 26, 2018
@guardrails
Copy link

guardrails bot commented Nov 26, 2018

⚠️ We detected security issues in this pull request:

Hard-Coded Secrets (1)

More info on how to fix Hard-Coded Secrets in General.

@90dy 90dy changed the title wip: refactor: start daemon refactor(rn): start daemon Nov 26, 2018
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
fix #624

Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Signed-off-by: Godefroy Ponsinet <godefroy.ponsinet@outlook.com>
Copy link
Member

@aeddi aeddi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@90dy 90dy merged commit 4d2e37e into berty:master Nov 27, 2018
@90dy 90dy deleted the refactor/gponsinet/rn-daemon-start-app-2 branch December 12, 2018 11:36
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.

Refactor start daemon Fix messages that are not updated in live (requires a reboot of the app)
3 participants