[ENG-3193] refactor: Refactor away from selectedOrdinal

[terencehh]

🔘 PR Type

What kind of change does this PR introduce?

• [x] Bugfix
• [ ] Enhancement
• [x] Code style update (formatting, local variables)
• [X] Refactoring (no functional changes, no API changes)
• [ ] Build related changes
• [ ] CI related changes
• [ ] Documentation content changes
• [ ] Other... Please describe:

📜 Background

There is a state bug where selectedOrdinal has improper state when on multiple tabs (it overwriting the current selectedOrdinal on the page). This PR introduces a fix by removing state management for selectedOrdinal, and instead always opting to look up the ordinal using the ordinalId in URL params, and finding the selected ordinal via react-query hooks (with caching)

Issue Link: ENG-3193 Context Link (if applicable):

🔄 Changes

• removed selectedOrdinal state actions, reducers, and setters
• updated all selectedOrdinal usage to call react-query hook which retrieves ordinal details via the ordinalId supplied in URL params

Impact:

• fixes state management bugs when on multiple tabs in send ordinals flow (sendOrdinal & confirmOrdinalTransaction screens)
• indirectly improves SEO since URL contains data now

🖼 Screenshot / 📹 Video

Send/Confirm Ordinals Screen Bug video

https://github.com/secretkeylabs/xverse-web-extension/assets/30040178/4699cbe7-4718-4a61-84b0-cdaa313a1412

Send/Confirm Ordinals Screen Fix video

https://github.com/secretkeylabs/xverse-web-extension/assets/30040178/ca6f21dc-aecf-4a55-943a-679429a84938

Recover Funds/Ordinals Screen Bug video

https://github.com/secretkeylabs/xverse-web-extension/assets/30040178/d010e5d2-b53c-43b3-a35a-4594a583e6c9

Recover Funds/Ordinals Screen Fix video

https://github.com/secretkeylabs/xverse-web-extension/assets/30040178/96f1a5ec-4db0-48b2-9b91-77d452f00f4e

✅ Review checklist

Please ensure the following are true before merging:

• [x] Code Style is consistent with the project guidelines.
• [x] Code is readable and well-commented.
• [x] No unnecessary or debugging code has been added.
• [x] Security considerations have been taken into account.
• [x] The change has been manually tested and works as expected.
• [ ] Breaking changes and their impacts have been considered and documented.
• [x] Code does not introduce new technical debt or issues.

[teebszet]

nice! so much cleaner. love it when we can remove code like this

[DuskaT021]

@terencehh there is a small conflict that needs to be resolved

[github-actions[bot]]

Test with build here: https://github.com/secretkeylabs/xverse-web-extension/actions/runs/8018139200

[DuskaT021]

@terencehh tested, noice, lgtm 👏