-
Notifications
You must be signed in to change notification settings - Fork 5
Liquid Mainnet and Testnet Mempool Proper Setup #51
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
base: master
Are you sure you want to change the base?
Conversation
…r updates Liquid Updates updates (feat): Updates for Liquid Mainnet, Testnet and Dockerization
c44d218 to
2735818
Compare
| @@ -0,0 +1,122 @@ | |||
| version: "3.7" | |||
|
|
|||
| services: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the previous one is for mainnet so this is for angornet?
| @@ -0,0 +1,75 @@ | |||
| <div class="container-fluid"> | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this doesn't exist already for the mainnet deployment?
| @@ -0,0 +1,58 @@ | |||
| import { Injectable } from '@angular/core'; | |||
| import { HttpClient } from '@angular/common/http'; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not liquid specific so what is this service exactly?
| isAngorEnabled() { | ||
| return this.env.ANGOR_ENABLED; | ||
| } | ||
| // Enable Angor for liquid networks specifically |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will this disable none liquid networks?
| testnet: true, | ||
| signet: false, | ||
| liquid: true, | ||
| liquidtestnet: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is all this needed? it is not liquid specific is it?
| ], | ||
| declarations: [ | ||
| MasterPageComponent, | ||
| AngorProjectsComponent, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again also here this angor already works on mempool so why do we need to define it here? it should be already defined no?
| import { Observable } from 'rxjs'; | ||
| import { AngorService, AngorProject, AngorStats } from '../../../services/angor.service'; | ||
|
|
||
| @Component({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there should be such a component as it is not liquid specific
|
@dangershony reverting these specific changes back and checking the status of the Liquid Mainnet. |
Liquid Mainnet and Testnet Proper Setup