Yu-Lin "Kirin" Chu

Results 28 comments of Yu-Lin "Kirin" Chu

> silent these annoying errors and give it an empty list for everything. OK! Thanks for the clarification.

Well, it seems like the dashboard has [enzyme](https://github.com/enzymejs/enzyme) as a [devDependency](https://github.com/hyperledger-cello/cello/blob/fad61c23635c32039bd6fd3a03f1e8b74983d9c1/src/dashboard/package.json#L109), which in turn depends on [cheerio](https://github.com/cheeriojs/cheerio). Five days ago, cheerio released a new version [1.1.0](https://github.com/cheeriojs/cheerio/releases/tag/v1.1.0), which upgrades its dependency...

> I didn’t find any usage of Enzyme in the dashboard, so maybe we can just remove it? @dodo920306 Well, I'm actually not sure. It's not my project in the...

Did you also enable *Rebuild open Merge Requests*?

> > Did you also enable _Rebuild open Merge Requests_? > > Hi [@dodo920306](https://github.com/dodo920306), Thank you for looking into it. I'm unsure where to find this option, 'Rebuild open Merge...

# Philosophy One key principle behind this PR is that unused code should be removed from the codebase. If a piece of code becomes relevant again in the future, it...

# Delivery 1. The Docker files now are moved into api-engine and dashboard so that in docker-compose files they can be built with context. # Side Effect 1. The original...

> @YoungHypo suggest we create a new branch for this refactor work. I just realized that I myself can also create a new branch in this repo. Therefore, I created...

The changes here have been split into 9 PRs: 1. #741 2. #742 3. #743 4. #744 5. #745 6. #746 7. #747 8. #748 9. #749

> No. Network should not be deleted as long as it contains any channel or node. However, currently nodes can't be deleted if it's the only orederer in a network.