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

[ABW-1340, ABW-1474, ABW-1476] Expand Dapp Details #473

Merged
merged 16 commits into from
May 10, 2023
Merged

Conversation

kugel3
Copy link
Contributor

@kugel3 kugel3 commented May 4, 2023

ABW-1340 - Show related resources in dApp Details
ABW-1474 - Only show canonical metadata in dApp Details
ABW-1476 - Add "Associated dApps" section to dApp details

Description

In dApp details there should be sections with fungible and non-fungible resources that are related to the dApp.

Notes

This PR is based on ABW-1400, branch wise, so needs to be rebased before merging.

Similarly to that PR, we might want to make the resources tappable, but currently this is blocked by the same reason as over there.

How to test

  • Connect to a dApp that has resources
  • Go to Settings -> Authorized dApps
  • Select the dApp
  • Scroll down and ensure that the resources are listed under their respective heading

Screenshot

Coming

PR submission checklist

  • I have tested account to account transfer flow and have confirmed that it works

Copy link
Contributor

@CyonAlexRDX CyonAlexRDX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

flatMap { .success(transform($0)) }
}

public func flatMap<NewValue>(_ transform: (Value) -> Loadable<NewValue>) -> Loadable<NewValue> {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition!

Base automatically changed from ABW-1400_persona-dApps to main May 5, 2023 13:08
@kugel3 kugel3 force-pushed the ABW-1340_dApp-details branch 5 times, most recently from b04b765 to d0413b1 Compare May 9, 2023 21:37
@kugel3 kugel3 changed the title [ABW-1340] Show Related Resources in Dapp Details [ABW-1340, ABW-1474, ABW-1476] Expand Dapp Details May 10, 2023
@kugel3 kugel3 merged commit f1806df into main May 10, 2023
@kugel3 kugel3 deleted the ABW-1340_dApp-details branch May 10, 2023 14:05
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants