Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Move network to app level #4733

Merged

Conversation

ishantiw
Copy link
Contributor

@ishantiw ishantiw commented Jan 23, 2020

Description

Move the network to application level for the reasons explained in #3358

Review checklist

@ishantiw ishantiw self-assigned this Jan 23, 2020
@ishantiw ishantiw added this to the Sprint 12 milestone Jan 23, 2020
@shuse2 shuse2 removed this from the Sprint 12 milestone Jan 23, 2020
@ishantiw ishantiw force-pushed the 3358-move_network_module_to_app_level branch from 7fefc31 to 1be645d Compare January 24, 2020 16:24
@ishantiw ishantiw changed the base branch from feature/move_chain_network to development January 24, 2020 16:27
@ishantiw ishantiw changed the base branch from development to feature/move_chain_network January 24, 2020 16:27
@ishantiw ishantiw marked this pull request as ready for review January 24, 2020 16:54
Copy link
Collaborator

@shuse2 shuse2 left a comment

Choose a reason for hiding this comment

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

Other than the naming, LGTM

Copy link
Contributor

@ManuGowda ManuGowda left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Question and comments.

framework/src/controller/controller.js Outdated Show resolved Hide resolved
framework/src/controller/network/network.js Outdated Show resolved Hide resolved
framework/src/controller/network/network.js Outdated Show resolved Hide resolved
framework/src/controller/network/network.js Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Jan 27, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
2.1% 2.1% Duplication

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

Successfully merging this pull request may close these issues.

4 participants