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

Enhanced Open Enterprise template #3

Open
4 tasks done
javieralaves opened this issue Nov 25, 2019 · 11 comments
Open
4 tasks done

Enhanced Open Enterprise template #3

javieralaves opened this issue Nov 25, 2019 · 11 comments

Comments

@javieralaves
Copy link
Contributor

javieralaves commented Nov 25, 2019

Description

Releasing a new template or enhancing Open Enterprise such that it permits an additional Tokens instance, allowing either of the two Tokens instances to be configured as defined by the organization.

Acceptance criteria

  • An additional Tokens app instance can be initiated
  • Either of the two Tokens app instances can be configured flexibly
  • Either of the two Tokens apps can be referenced to govern over the Voting or Dot Voting app
  • Agent app is available as an optional app for install

What's not to be included in this epic:

  • Any new Help components, which need to be further defined in a different epic

Product specs

Available on Google Drive.

@javieralaves javieralaves added this to the Sprint 3 milestone Nov 25, 2019
@javieralaves javieralaves added onboarding product spec A product spec is needed labels Nov 25, 2019
@javieralaves javieralaves self-assigned this Nov 27, 2019
@javieralaves javieralaves removed the product spec A product spec is needed label Nov 29, 2019
@javieralaves
Copy link
Contributor Author

javieralaves commented Nov 29, 2019

@stellarmagnet Ready for your review. This latest iteration covers perhaps the simplest solution to the two-Tokens template design, which would be to add a Help icon for contextual information. I wasn't able to find the right fit for something different, for now. I've also defined the Tokens initialization parameters, similar to what you defined a few weeks ago.

@stellarmagnet
Copy link
Collaborator

Hello Javi! I added some comments to the Figma. One idea is we can simply bring back one of the previous designs you had, but just reword it so the user is choosing between 1 or 2 token org. I'm just not sure if the current way with the additional "Switch" for Tokens would be intuitive...

I think the switch does make sense for adding the Agent. I feel we should add a help icon there too, as first time users may not know the benefits of adding the agent.

I added a few comments to the spec, some things we need to clarify with development. I think we have to allow the user to select which token is tied to Voting or Dot Voting. We are going to be assuming here that only one voting app is needed (e.g. they are only mapping it to one token) although we are allowing mixed governance between Voting and Dot Voting.

@javieralaves javieralaves added the product spec A product spec is needed label Dec 1, 2019
@javieralaves javieralaves changed the title Reputation template release Enhanced Open Enterprise template Dec 1, 2019
@javieralaves javieralaves removed the product spec A product spec is needed label Dec 3, 2019
@javieralaves
Copy link
Contributor Author

@stellarmagnet This is ready for product review!

@javieralaves javieralaves removed their assignment Dec 3, 2019
@Quazia
Copy link

Quazia commented Dec 3, 2019

@ottodevs is going to review Peters work, and handle the client implementation to the spec. For the contract @PeterMPhillips can start by moving the testing from Otto's branch over to his template implementation and add any additional tests needed. From there we just need to confirm that the contract work is in sync with the needs from the Gdocs specification.

@stellarmagnet
Copy link
Collaborator

@javieralaves looks good to me, there are just a few design comments in the Figma that I think shouldn't take that long to resolve. Moving this to technical review.

@javieralaves
Copy link
Contributor Author

@stellarmagnet Figma and specs updated, this is ready to go!

@javieralaves javieralaves removed their assignment Dec 4, 2019
@ottodevs
Copy link
Member

ottodevs commented Dec 6, 2019

Hi @javieralaves ! While implementing the UI for this, closely following your figma design, I detected an unusual size for the 1 token asset, this one:
image

Just removing the shirt pattern of the sitting person in the illustration, the size goes from 2.2MB to 8KB, so I was wondering if we might want to optimize that asset.

I also detected unusual sizes in other open-enterprise assets using the same pattern, specifically these ones:
image

Basically the empty state card illustrations for those 3 apps.

It is probably barely noticeable with the modern fast connections, but I think we still could try to optimize that, even for processing those files from the code perspective it may result slow to manage and we seem to be defeating the vector graphics advantages, as these images are actually heavy png files embedded into an svg pattern

If you find it is not evident how to improve the assets, I can try looking into any cli or online tool for image optimization, just ping me with your thoughts 👍

@javieralaves
Copy link
Contributor Author

Hey @ottodevs, this is definitely of relevance, thank you for point this out! I'll keep a better eye on this aspect in the future and send you optimized assets today or tomorrow.

@ottodevs
Copy link
Member

Hey @ottodevs, this is definitely of relevance, thank you for point this out! I'll keep a better eye on this aspect in the future and send you optimized assets today or tomorrow.

FYI during last weekend I was playing a bit with these assets and I manage to reduce their size by 10 times by following this methodology:

  • Copy the embedded png pattern (encoded into base64) and use online tool to decode the actual png image
  • Use another online tool to resize and optimize png images, reducing the size over 90%
  • Encode again into a base64 string
  • Paste again in the svg markup pattern layer

Of course if we do this during design phase, those steps would be probably unnecessary, but maybe if you cannot figure out a better way, it might do the trick for now.

Also looking on the A1 take on this, I realized they just use strongly optimized png files for the aragon client instead of using svg at all... but I guess -and this is a personal opinion- that svg is a better format for these kind of assets, because the scalability for high density resolutions.

@javieralaves
Copy link
Contributor Author

Thanks @ottodevs! And sorry that I was unable to get these to you earlier. I agree with your opinion about using svg over png as a file format. I'll make sure that either way future graphic assets are as optimized for deployment as they can be.

@javieralaves javieralaves modified the milestones: Sprint B2, Sprint B3 Dec 16, 2019
@stellarmagnet stellarmagnet modified the milestones: Sprint B3, Sprint B2 Dec 16, 2019
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