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

Identifying signature requests initiated from profile app #855

Closed
Schwartz10 opened this issue Jul 1, 2019 · 5 comments
Closed

Identifying signature requests initiated from profile app #855

Schwartz10 opened this issue Jul 1, 2019 · 5 comments

Comments

@Schwartz10
Copy link
Contributor

When the signer panel opens with a signature request from the profile app, the AppInstanceLabel component displays “unknown” because the profile application is not tied to a smart contract.

I wanted to check and see if we have a specific design request to handle this situation, and/or if there are any specific implementation suggestions. I think (for now) it makes sense to create a custom AppInstanceLabel component for Aragon apps without smart contracts (what should we call that?), and conditionally render the InstanceLabel from the SignerPanel depending on the type of app that requested the signature.

Do people have thoughts on this? If this seems like low priority, I’m happy to adjust the current AppInstanceLabel component to handle smart contract-less apps as well.

@luisivan
Copy link
Contributor

luisivan commented Jul 2, 2019

I’m happy to adjust the current AppInstanceLabel component to handle smart contract-less apps as well.

Yeah I think this would make more sense than having a different component. Thoughts @bpierre @sohkai?

@bpierre
Copy link
Contributor

bpierre commented Jul 2, 2019

Yes I agree, the profile app is the only internal app to use the signer panel for now, so it makes sense to just adjust the AppInstanceLabel for it.

@sohkai
Copy link
Contributor

sohkai commented Jul 2, 2019

cc @dizzypaty This is the "Unknown" label (because Profiles is not attached to a contract):

Screen Shot 2019-07-02 at 6 35 59 PM

@luisivan
Copy link
Contributor

luisivan commented Jul 3, 2019

Great, then @Schwartz10 feel free to open a PR and @bpierre will review it!

@sohkai
Copy link
Contributor

sohkai commented Mar 18, 2020

Closing for now; the Profiles app is not on a timeline to be included in the client.

@sohkai sohkai closed this as completed Mar 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants