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

Feature - Add Tezos Profiles Verifications for Creators #868

Merged
merged 1 commit into from
Jun 12, 2021
Merged

Feature - Add Tezos Profiles Verifications for Creators #868

merged 1 commit into from
Jun 12, 2021

Conversation

obstropolos
Copy link
Contributor

Description

This change adds a badge to a creator's page if they have an existing Tezos Profile created using the Tezos Profiles service. Additionally, it adds an alias and Twitter badge from a user's Tezos Profile if the user does not already have one from the existing TzKT authentication service. The data is currently being pulled from the Tezos Profiles API.

Tezos Profiles and all of its components are completely open-source (including the indexer and API) and can easily be rehomed to any other server. Users choose what URLs are resolved to fetch and check their verifications and they can update their own smart contracts to new URLs at any time. This uses TZIP-024 to ensure interoperability and protect user privacy.

Progress

  • Add the Tezos Profiles API
  • Add a badge for an existing Tezos Profile
  • Add a Twitter badge if no account is found using the TzKT service
  • Add an alias if none is found using the TzKT service

A demo deployment of the service with the add-on can be found here.

The goal of this integration is to begin to add additional creator verifications to HEN profiles and help prevent copyminting.

Testing

Screen Shot 2021-06-11 at 3 26 22 PM

Screen Shot 2021-06-11 at 3 27 15 PM

Future Goals

Eventually, we wish to augment the service with additional credentials for artists in order to better verify themselves. These credentials can include additional social profile verifications and even verifications from other blockchains. Additionally, we would like the HEN community to host their own API and indexer for Tezos Profiles, which only requires a modest server.

Adds:
- Twitter handle if not present in TzKT already; and
- use alias and add logo for self-certified profile.
@crzypatchwork
Copy link
Contributor

crzypatchwork commented Jun 11, 2021

I would suggest to use alias in a different way to make profiles more accessible

for example: hicetnunc.xyz/crzypatchwork is way more accessible then hicetnunc.xyz/tz/tz1...

@veqtor
Copy link
Contributor

veqtor commented Jun 11, 2021

I would suggest to use alias in a different way to make profiles more accessible

for example: hicetnunc.xyz/crzypatchwork is way more accessible then hicetnunc.xyz/tz/tz1...

I would say yes, in the long run but this would need backend and more complex contract, also, is it really up to tzp to decide who gets which alias?

@crzypatchwork crzypatchwork merged commit 061ca75 into hicetnunc2000:develop Jun 12, 2021
@crzypatchwork
Copy link
Contributor

crzypatchwork commented Jun 12, 2021

Screenshot from 2021-06-12 08-59-04

not sure if the tzprofile badge is working from the verification process I went through

https://tzkt.io/KT1SeUWpeioMYNaHZCwzUv5fKioLGd3y87Vw/operations/

@sbihel
Copy link
Contributor

sbihel commented Jun 12, 2021

Looks like we have a bug on our side for accounts with many contracts, sorry about that.

@sbihel
Copy link
Contributor

sbihel commented Jun 12, 2021

It has been fixed.

@obstropolos
Copy link
Contributor Author

Hi @crzypatchwork, thanks for giving us the heads up. For accounts that had many contracts such as yours, the indexer was not behaving as expected. We have switched indexer endpoints and added pagination, which has addressed this.

spruceid/tzprofiles#99

You should now be able to see the icon and other data here:
https://a1b3165a.hicetnunc.pages.dev/tz/tz1Y1j7FK1X9Rrv2VdPz5bXoU7SszF8W1RnK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants