-
Notifications
You must be signed in to change notification settings - Fork 13
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
Make marketplace configurable by the core #942
Conversation
@@ -7,6 +7,9 @@ MESG_LOG_LEVEL=${MESG_LOG_LEVEL:-debug} | |||
MESG_LOG_FORCECOLORS=${MESG_LOG_FORCECOLORS:-false} | |||
VERSION=local | |||
LDFLAGS="-X 'github.com/mesg-foundation/core/version.Version=$VERSION'" | |||
LDFLAGS+=" -X 'github.com/mesg-foundation/core/config.EnvMarketplaceEndpoint=https://ropsten.infura.io/v3/6115ae2531f64c04a9a392cf500e5fbe'" |
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.
We can set it directly in the mesg.yaml of the service, in that case everone can deploy and use it without dev-core script.
EDIT:
Those values were in the mesg.yaml, what is the reason to move it here? We can left it in mesg.yaml and overwrite via config if needed
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 problem is in the file config/services.go
. It always replace the env.
It could create the env map dynamically only if the variable contains something...
It could be improved later on. Let's merge it this way for now.
I added the mainnet data 👍 See the deployed marketplace: https://etherscan.io/address/0x0c6e8d0ec4770fda8a56cd912392d2ff14822952 |
We need a way to use the marketplace in dev (ropsten) and prod (on the mainnet)
@NicolasMahe please update
scripts/build-core.sh
when you have the contract on the mainnet