Feature/front end data renaming #376
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DemogorGod
requested review from
a team,
ccali11,
shanejearley and
hawyar
and removed request for
a team
|
@DemogorGod the links would be invalid until testnet launch anyways, so you can put that off for this PR. |
shanejearley
approved these changes
Jul 19, 2023
There was a problem hiding this comment.
@DemogorGod Perfect changes.
@elizyoung0011 Time to add tooltips? Could make an issue if you want Steve to start that.
|
Yep- I’ll creat a tooltip issue with text. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey guys, just changed some of the front-end stuff to match with the requested stuff on the table side and staking component side as well. my commits basically describe everything I did. Let me know if any other changes are needed and I will adjust them in future prs. Enjoy the video!
@elizyoung0011 currently the links on the table do not connect to any external websites ( I believe I was told not to do it before ) so if you want to confirm that, that is the intended user flow on our website is to take users onto other websites and not create modals/pop ups that shows the information that other websites have, then I can do that for sure. I will also need a format on how to connect to those sites using a specific tx hash or other variables. Thanks!!
@ccali11 There are some spots on the fe that require a method that fetches the current staked total by wallet address if you can just create me a method on the backend in one of your composables for me i can connect it to all of the places that it is needed. Thanks!!
Screen.Recording.2023-07-18.at.10.01.51.PM.mov