Skip to content
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

Added The Galgos NFT project #163

Merged
merged 1 commit into from
Aug 15, 2021

Conversation

thegalgos
Copy link
Contributor

No description provided.

@katomm
Copy link
Member

katomm commented Jul 30, 2021

Hello,
this PR has been open for almost 2 weeks and I would like to give you a little feedback about it here. It probably didn't get enough "traction" among reviewers. Please note that showcase projects need to provide a unique value from existing showcase items and need to have enough community reputation.

My personal suggestions:
Close this PR, try to improve it, try to get more traction in the community and come back later. Maybe you could also find other parts in the portal where you can contribute.

References:
The current guidelines: https://github.com/cardano-foundation/developer-portal/edit/staging/src/data/showcases.js

Discussions about guidelines: https://github.com/cardano-foundation/developer-portal/discussions

@katomm katomm added the showcase Indicates a PR/issue on showcase label Aug 2, 2021
@thegalgos
Copy link
Contributor Author

Hi @katomm, how you measure enough reputation from the community?

@katomm
Copy link
Member

katomm commented Aug 3, 2021

Hi @katomm, how you measure enough reputation from the community?

Community reputation obviously cannot be measured in numbers.

Copy link
Member

@katomm katomm left a comment

Choose a reason for hiding this comment

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

LGTM

@thegalgos
Copy link
Contributor Author

Here is Charles asking why this is not yet merged ;)

Anybody can help? The #CNFT community is asking for this. Thanks!

0008

@thegalgos
Copy link
Contributor Author

Our project is different from the rest as we are developing with smart-contracts to add functionality to our NFTs. Obviously we are waiting for the deployment on the main-net. But for example, these are galgos/dogs and they have a default given name. If you want, you could change its name to the one that you like, this way the NFT is updated based on holder preferences. We burn the token and we mint the same but with updated metadata. This will make them more unique. Another example, we are working a game so your galgo is something that will evolve based on your actions. You can think of a play-to-earn game. If you want more details please let me know, thanks!

@Mercurial Mercurial merged commit a9b4035 into cardano-foundation:staging Aug 15, 2021
Mercurial added a commit that referenced this pull request Aug 23, 2021
…in (#260)

* Bump color-string from 1.5.4 to 1.6.0

Bumps [color-string](https://github.com/Qix-/color-string) from 1.5.4 to 1.6.0.
- [Release notes](https://github.com/Qix-/color-string/releases)
- [Changelog](https://github.com/Qix-/color-string/blob/master/CHANGELOG.md)
- [Commits](Qix-/color-string@1.5.4...1.6.0)

---
updated-dependencies:
- dependency-name: color-string
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump prismjs from 1.23.0 to 1.24.1

Bumps [prismjs](https://github.com/PrismJS/prism) from 1.23.0 to 1.24.1.
- [Release notes](https://github.com/PrismJS/prism/releases)
- [Changelog](https://github.com/PrismJS/prism/blob/master/CHANGELOG.md)
- [Commits](PrismJS/prism@v1.23.0...v1.24.1)

---
updated-dependencies:
- dependency-name: prismjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump path-parse from 1.0.6 to 1.0.7

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Add showcase guideline proposed in #176 (#235)

* Added The Galgos NFT project (#163)

* Add Python Module (#220)

* Add Python Module

* Add mention of cardano-wallet backend

* Remove tags

* Remove getstarted link

* fix urls and native token sentence (#240)

* fix: backslash missing from code example (#246)

* removed duplicate for copy command (#244)

* IO Fix (#245)

Co-authored-by: Priyank

* Update get started with developer community (#249)

* august developer spotlight article mercury (#256)

Co-authored-by: Kevin Cislak <kevin.cislak@cardanofoundation.org>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tommy <31965230+katomm@users.noreply.github.com>
Co-authored-by: thegalgos <86036870+thegalgos@users.noreply.github.com>
Co-authored-by: Michał Sałaban <michal@salaban.info>
Co-authored-by: cislakk <kjcislak@gmail.com>
Co-authored-by: Ari V <11803028+arivictor@users.noreply.github.com>
Co-authored-by: Jackson Barnes <30503489+atx-barnes@users.noreply.github.com>
Co-authored-by: Kevin Cislak <kevin.cislak@cardanofoundation.org>
katomm added a commit that referenced this pull request Sep 7, 2021
* Bump color-string from 1.5.4 to 1.6.0

Bumps [color-string](https://github.com/Qix-/color-string) from 1.5.4 to 1.6.0.
- [Release notes](https://github.com/Qix-/color-string/releases)
- [Changelog](https://github.com/Qix-/color-string/blob/master/CHANGELOG.md)
- [Commits](Qix-/color-string@1.5.4...1.6.0)

---
updated-dependencies:
- dependency-name: color-string
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump prismjs from 1.23.0 to 1.24.1

Bumps [prismjs](https://github.com/PrismJS/prism) from 1.23.0 to 1.24.1.
- [Release notes](https://github.com/PrismJS/prism/releases)
- [Changelog](https://github.com/PrismJS/prism/blob/master/CHANGELOG.md)
- [Commits](PrismJS/prism@v1.23.0...v1.24.1)

---
updated-dependencies:
- dependency-name: prismjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump path-parse from 1.0.6 to 1.0.7

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Add showcase guideline proposed in #176 (#235)

* Added The Galgos NFT project (#163)

* Add Python Module (#220)

* Add Python Module

* Add mention of cardano-wallet backend

* Remove tags

* Remove getstarted link

* fix urls and native token sentence (#240)

* fix: backslash missing from code example (#246)

* removed duplicate for copy command (#244)

* IO Fix (#245)

Co-authored-by: Priyank

* Update get started with developer community (#249)

* august developer spotlight article mercury (#256)

Co-authored-by: Kevin Cislak 

* Fix Spelling of Address (#258)

Change "Adress" to "Address"

* Fix spelling of 'metadata' (#261)

Changed "metdata" to "metadata"

* quickfix typo (#262)

* fix grammatical error

* fix some typos and grammar

quickfixes

* fix repetition of 'the' (#263)

* removed unsigned_algorithms (#264)

* Fix typo in portal style guide (#269)

* Update testnet page (Alonzo) (#280)

* Fix testnet faucet iFrame

* Remove Shelley and Mary references, rename Daedalus testnet

* Make the testnet explorer easier to find, update text

* Fix indentations

* Update links

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Clark Alesna 
Co-authored-by: thegalgos 
Co-authored-by: Michał Sałaban 
Co-authored-by: cislakk
Co-authored-by: Ari V 
Co-authored-by: Jackson Barnes 
Co-authored-by: Kevin Cislak 
Co-authored-by: Charles Jenkins 
Co-authored-by: Pedro Moreira 
Co-authored-by: Daniel Koman
Co-authored-by: Bb 
Co-authored-by: Alexander Watanabe 
Co-authored-by: Ben Bowen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
showcase Indicates a PR/issue on showcase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants