x
xenosgeck
Discord
Joined
icon image
Celo Community
87.5 REP
4 Feedbacks
1 Propuesta

Propuestas

x
xenosgeck
87.5 REP
Enviado

CeloDomain is a simple domain name service on Celo blockchain. You can create your own domain name (with .celo as the top domain) as a NFT. The price for minting a name is 0.01 CELO.

16Puntos
2 Feedbacks

FEEDBACK

A
Arome8240
Enviado

This dapp is a decentralized marketplace to buy and sell land on the celo blockchain. Any user can upload a token and also sell it. Once token is uploaded it is transferred to the smart contract address, then any user can buy it. This is to ensure every land made can be digitized as an NFT to enable the distribution of land decentralized

14Puntos
2 Feedbacks
x
xenosgeck
87.5 REP
Feedback

Hi, Nice submission. Code looks clean.

I have a couple of feedbacks:

  1. In the contract, I think it makes sense not to transfer the recently minted token to the marketplace contract directly. After minting the token, the user then can decide to sell it or not.
  2. When a user sells his or her token, the website shows that the user can buy their own token back. CELO tokens will be sent to the marketplace and then back to their wallet. In the PR, I proposed to create a new function "unSellLand" to cancel the sale and transfer the NFT back to the owner.
  3. Small updated in the frontend to allow users to "cancel sale" their NFT.

Hope it is helpful. Overall nice project.

37.5 cUSD
37.5 REP
user-avatar
x
xTargetz
Enviado

Daisy's One Stop Shop is an NFT minting website to make your own NFTs. You can mint, and sell on this website. I could not get GitHub pages to work correctly so I deployed the app to Netlify. https://628285e3ec0e6b009c9c1ca4--fastidious-mousse-b944f8.netlify.app/

7Puntos
2 Feedbacks
x
xenosgeck
87.5 REP
Feedback

Hi, I like the UI. Here are some improvements.

  1. Removed node_modules and updated .gitignore
  2. Cleanup unused variables
  3. Show "Please connect wallet" if the user has not connected the wallet yet.
  4. Fix typo cUSD -> CELO, Creater -> Creator
  5. User should not be able to buy their own NFTs. Disable "Buy" button.
  6. Updated README
37.5 cUSD
37.5 REP
j
jefedcreator
Enviado

Abstrakt is a nft creation dApp for creating your favourite pictures on the blockchain. connect your wallet, and mint nfts to yourself on the celo blockchain. each abstract metadata is stored onchain and viewed when users login with their wallets.

14Puntos
2 Feedbacks
x
xenosgeck
87.5 REP
Feedback

Hi, overall good submission. I like how the landing page look like. I submitted a pull request with these changes:

  1. Added a link in the landing page.
  2. Updated the "Create" button to "NFT collections". Notice that there is also a + button in the /create page. I changed the theme to light to make it obvious for users.

Hope you like it!!

12.5 cUSD
12.5 REP
user-avatar
r
richiemike
Enviado

Doc-nft-minter is a unique Dapp where the owner can use the Dapp to mint documents into a form of NFT to show other parties as evidence of ownership of properties. It also has a transfer function that can facilitate the transfer of the NFT Document to another address. But only the owner of the contract can see the transfer button or to mint button.

16Puntos
4 Feedbacks
x
xenosgeck
87.5 REP
Feedback

Hi, I think you haven't pushed your latest changes to your Github repository. I have tried to fork your repository and ran your code on my local machine, and I noticed that there's some differences on the frontend (i.e. there is no 'transfer' button on my local machine although I am the owner of the contract).

The contract implementation looks okay. I would add an extra check to first check if the NFT belongs to the owner or not, before transferring the NFT to another address. As specified, only the owner of the contract can do that

Dacade es una plataforma de código abierto y se creó en colaboración con múltiples contribuyentes. Visita el repositorio para empezar a contribuir.

Impressum & Política de Privacidad