Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: prep component minting #60

Merged
merged 5 commits into from
Aug 25, 2023
Merged

chore: prep component minting #60

merged 5 commits into from
Aug 25, 2023

Conversation

DavidMinarsch
Copy link
Contributor

No description provided.

X-
X-"contract/valory/mech/0.1.0"
X-"contract/valory/erc20/0.1.0"
X-"contract/valory/market_maker/0.1.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This name could collide with some components in the market-creator repository (e.g., agent/valory/market_maker/0.1.0, service/valory/market_maker:0.1.0)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Are they the same or different contracts?

Copy link
Collaborator

Choose a reason for hiding this comment

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

They are different.

@@ -0,0 +1 @@
{"name":"contract/valory/market_maker:0.1.0","description":"Fixed product market maker contract","code_uri":"ipfs://bafybeighn4o263yxjxc46n5rckfnonwtyyhvjmegmkrqraixvswuyhcsp4","image":"ipfs://bafybeievzsyvjc3oyuwrnlceionzkdbhydoripjeuvlngbkhahjix2c7vq","attributes":[{"trait_type":"version","value":"0.1.0"}]}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why "fixed"?

Copy link
Collaborator

Choose a reason for hiding this comment

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

mints/06.json Outdated
@@ -0,0 +1 @@
{"name":"skill/valory/decision_maker_abci:0.1.0","description":"ABCI application for common apps.","code_uri":"ipfs://bafybeigircx36tx3nw3r6dgym357e3wyzabb7y7zhahhttwbkgnxinsgre","image":"ipfs://bafybeiasruosr2t2qzv4iqoaj3l5idztygk5q36khovdfrdwzbddwmi3km","attributes":[{"trait_type":"version","value":"0.1.0"}]}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This description of the skill is not a good one

Copy link
Collaborator

Choose a reason for hiding this comment

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

Improved on #63.

@DavidMinarsch DavidMinarsch merged commit 134457b into main Aug 25, 2023
@DavidMinarsch DavidMinarsch deleted the mints branch August 25, 2023 18:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants