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

Teia UI update #449

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

ryangtanaka-org
Copy link
Contributor

@ryangtanaka-org ryangtanaka-org commented Feb 2, 2025

  • added donation button to the treasury, a quipuswap link to convert xtz into TEIA

  • fixed incorrect contract for the treasury balance display - please confirm that (KT1RRLSxpsiKpnVZQsBGCqG9HFBXbPBtUYBD) is correct

  • added core parts of the documentations into the top right hand corner menu items (the text is in lang/en if edits are needed)

  • minor fixes in the feeds dropdown menu on the top left

  • added subtitles to the swap tab so it's easier to see

  • added explanations to the DAO and polling pages for more clarity

  • FAQ page updates and added explanation for the copyright feature

  • various UI fixes throughout the site

@Zir0h
Copy link
Contributor

Zir0h commented Feb 4, 2025

Why did you add the markdown-to-jsx and fetch stuff in for example https://github.com/teia-community/teia-ui/blob/4df6007632a298632b63016bf00b90cdabdd0266/src/pages/codeofconduct/index.jsx ? The markdown rendering should also just render if you imported it the same way as for example https://github.com/teia-community/teia-ui/blob/main/src/pages/about/index.jsx

@Zir0h
Copy link
Contributor

Zir0h commented Feb 4, 2025

I think treasury balance should be taken from KT1J9FYz29RBQi1oGLw8uXyACrzXzV1dHuvb (core team multisig) instead.

// and KT1J9FYz29RBQi1oGLw8uXyACrzXzV1dHuvb should also be the donation target address I think

src/components/header/feed_toolbar/FeedToolbar.jsx Outdated Show resolved Hide resolved
src/components/header/feed_toolbar/FeedToolbar.jsx Outdated Show resolved Hide resolved
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.

2 participants