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

Staking Page Missing in Polkadex UI After Latest Changes #10505

Closed
nuel77 opened this issue Apr 25, 2024 · 20 comments
Closed

Staking Page Missing in Polkadex UI After Latest Changes #10505

nuel77 opened this issue Apr 25, 2024 · 20 comments

Comments

@nuel77
Copy link
Contributor

nuel77 commented Apr 25, 2024

Hello.

I wanted to bring up an issue that we've noticed in the Polkadex UI after the recent changes from PR #10498 on the polkadot-js/apps repository.

It seems that the staking page section is now missing in Polkadex UI following these updates. I understand that there was a significant change introduced for pallet-staking in the new version, which might have caused this discrepancy.

My main question is whether the apps-ui team has any plans for providing backwards compatibility to address this issue, or if the fix requires upgrading to the latest version of substrate. Understanding the roadmap for resolving this would be immensely helpful for our next steps.

Looking forward to hearing your insights on this matter!

apps-ui central domain(staking page missing)
image

apps-ui ipfs doiman (still working)
image

@kdembler
Copy link
Contributor

Joystream is facing the same issue (Redirecting from route "/staking" to "/explorer", missing the following APIs: ["query.staking.erasStakersOverview"]) due to older staking runtime pallet. Hopefully this page could be made backwards compatible, otherwise we will need to host old version of apps ourselves.

@kdembler
Copy link
Contributor

@jacogr @TarikGul

@TarikGul
Copy link
Member

My main question is whether the apps-ui team has any plans for providing backwards compatibility to address this issue, or if the fix requires upgrading to the latest version of substrate. Understanding the roadmap for resolving this would be immensely helpful for our next steps.

First apologies on any issues this causes. Usually speaking I would aim from the start to ensure backwards compatibility but this specific problem with the new staking changes was a pretty wide net that extended from the @polkadot/api-derive package all the way to apps. So it wasn't as simple as providing compatibility in just the UI. Also it's hard to track every chain and what they use, especially when things become urgent.

The compatibility on the api level is the hardest here, as that is where all the observable calls, aggregation, and structuring happens for all the information that the staking UI leverages.

Hopefully this page could be made backwards compatible, otherwise we will need to host old version of apps ourselves.

This is never a bad idea.

That all being said, I am OOO for the next week (including today), but when back I will try to look into potential solutions to bring this compatibility solution to a more manageable and maintainable path moving forward. Maybe the best most reasonable solution (for now) is bringing compatibility to the api-derive staking level first, and just make a copy of the old staking page, call it staking-legacy, and alias the text to be staking in the UI so the experience feels the same.

@marc-aurele-besner
Copy link

marc-aurele-besner commented Apr 28, 2024

In the meantime, anyone can use the Old IPFS CID 😄

@PdexFan
Copy link

PdexFan commented Apr 29, 2024

In the meantime, anyone can use the Old IPFS CID 😄

I can confirm that this solution is not working for Polkadex Main Network.

@marc-aurele-besner
Copy link

marc-aurele-besner commented Apr 30, 2024

In the meantime, anyone can use the Old IPFS CID 😄

I can confirm that this solution is not working for Polkadex Main Network.

I apologize if this solution doesn't help you. An ex-colleague had asked me how to find an old IPFS version of the app due to issues with the current app affecting him on another network, so I thought I would share this link here as well.

For reference, see the CID Log History if needing older CID

@TarikGul
Copy link
Member

TarikGul commented May 6, 2024

@nuel77 Just as a heads up I am now tackling this along with fixing staking for the relay chains simultaneously. Hopefully this will all be fixed on the API level today, then I can start tackling the UI bits.

@nuel77
Copy link
Contributor Author

nuel77 commented May 6, 2024

Thanks a lot for the update @TarikGul , looking forward to it 🥂

@MoyStell74
Copy link

Good morning everyone, I'm in the same situation, I can't see my goal on the mainet page polkadot.js polkadex network....I trust the team for a prompt update. We are aware
Success

@TarikGul
Copy link
Member

TarikGul commented May 7, 2024

The basis for backwards compatibility was merged into the API yesterday polkadot-js/api#5868, I just have one more porblem to tackle in the API to fix for claimed rewards with the new staking pallet then the changes will be released and reflected in apps.

@MoyStell74
Copy link

La base para la compatibilidad con versiones anteriores se fusionó en la API ayer polkadot-js/api#5868 . Solo tengo un problema más que abordar en la API para solucionar las recompensas reclamadas con la nueva paleta de apuestas, luego los cambios se publicarán y se reflejarán en las aplicaciones. .

I have been using the polkadex mainnet in polkadot.js for staking since 2021 and for about 3 weeks I have not been able to see the staking section on the network.
I can't make validator changes
I can't add more pdex
I can't withdraw
I can only make transactions with my rewards.
Can you tell me if this is going to be updated or is there another way to enter?
please
thank you

@TarikGul
Copy link
Member

TarikGul commented May 9, 2024

@MoyStell74 Yes, I am currently working on it as we speak: #10553

@MoyStell74
Copy link

@MoyStell74Sí, actualmente estoy trabajando en ello mientras hablamos: #10553
THANK YOU

@TarikGul
Copy link
Member

TarikGul commented May 9, 2024

@MoyStell74 If you really need to access the staking page for polkadex you can just run apps locally, at the commit before the changes.

The commit is ca3f368

You just need to pull down apps, and run the following.

$ git checkout ca3f368d47d381e128435ebb6dc587b704940bd1
$ yarn install && yarn start

Then go to http://localhost:3000/?rpc=wss%3A%2F%2Fpolkadex-mainnet-rpc.dwellir.com#/staking

And it will work as it previously did.

@nuel77
Copy link
Contributor Author

nuel77 commented May 9, 2024

La base para la compatibilidad con versiones anteriores se fusionó en la API ayer polkadot-js/api#5868 . Solo tengo un problema más que abordar en la API para solucionar las recompensas reclamadas con la nueva paleta de apuestas, luego los cambios se publicarán y se reflejarán en las aplicaciones. .

I have been using the polkadex mainnet in polkadot.js for staking since 2021 and for about 3 weeks I have not been able to see the staking section on the network. I can't make validator changes I can't add more pdex I can't withdraw I can only make transactions with my rewards. Can you tell me if this is going to be updated or is there another way to enter? please thank you

@MoyStell74 you can also try this which is an a old version of the apps UI hosted by the PolkaGate team. if you are facing any emergency use case.

@TarikGul
Copy link
Member

TarikGul commented May 9, 2024

PR is up and basically donte to fix the missing staking page: #10557

The tab is the same the only difference is the routing which is now legacy-staking

@TarikGul
Copy link
Member

TarikGul commented May 9, 2024

Feel free to test it locally if you would like!

@TarikGul
Copy link
Member

This is now fixed in the live version!

@nuel77
Copy link
Contributor Author

nuel77 commented May 10, 2024

Thanks alot @TarikGul 👍

@polkadot-js-bot
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.

@polkadot-js polkadot-js locked as resolved and limited conversation to collaborators May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants