Skip to content
This repository has been archived by the owner on Feb 18, 2022. It is now read-only.

Add TNS support #210

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jormungandr12
Copy link

Changes

  • Introduce helper field on Field interface to show a subtle note about the resolved address.

Screen_Shot_2564-11-08_at_17 21 56

  • Add TNS name resolver to 3 pages:
    • Send token (src/post/hooks/wallet/useSend.ts)
    • Community Pool Spend Proposal (src/post/hooks/gov/usePropose.ts)
    • Sign-in with address (src/hooks/auth/useSignInWithAddress.ts)
  • Since the validation function in useForm only supports synchronous function, TNS name resolution logic is done outside the useForm scope. Hence, useTns hook is introduced.
  • Add TNS name error This name has not yet been configured and other languages translation.

Test cases

  • Fill in blossom.ust, it should resolve to an address terra1...9hthwzp (Like above image)
  • Fill in axfkcsnjr.ust, it should show This name has not yet been configured validation error message.

Screen_Shot_2564-11-08_at_17 30 24

Expected Behavior

  • Send token
    station-wallet

  • Community Pool Spend Proposal
    station-gov

  • Sign-in with address
    station-connect

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

Successfully merging this pull request may close these issues.

1 participant