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

docs: add elevator pitch #59

Merged
merged 6 commits into from
Feb 23, 2023
Merged

docs: add elevator pitch #59

merged 6 commits into from
Feb 23, 2023

Conversation

juliangruber
Copy link
Member

No description provided.

@juliangruber juliangruber mentioned this pull request Feb 23, 2023
6 tasks
Copy link
Member

@bajtos bajtos left a comment

Choose a reason for hiding this comment

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

Great idea 👍🏻

README.md Outdated Show resolved Hide resolved
@juliangruber juliangruber enabled auto-merge (squash) February 23, 2023 13:52
Copy link
Member

@bajtos bajtos left a comment

Choose a reason for hiding this comment

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

On the second thought, I am no longer sure this is a good idea.

We already have an elevator pitch at the top, above the picture.

Zinnia is a runtime for Filecoin Station modules.

It provides a sandboxed environment to execute untrusted code on consumer-grade
computers.

I feel your proposal is adding somewhat duplicated content.

Can you please rework it to remove this redundancy?

> is the runtime for
> [Station](https://github.com/filecoin-station/filecoin-station) modules. It
> allows anyone - even untrusted parties - to deploy modules to Stations
> wordwide.

<div align="center">
<img src="https://s.yimg.com/uu/api/res/1.2/WtLPXqGgiUashZzP.J4drw--~B/Zmk9ZmlsbDtoPTU4Mzt3PTg3NTthcHBpZD15dGFjaHlvbg--/https://o.aolcdn.com/hss/storage/midas/229be0287167454b558989b2e29221d8/203272974/zinnias-success.jpg.cf.jpg" width="50%" />
Copy link
Member

Choose a reason for hiding this comment

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

Maybe it's time to remove this image now that we have enough content in this file?

Copy link
Member Author

Choose a reason for hiding this comment

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

what about we either remove or replace it with a midjourney one, in a separate PR?

@juliangruber juliangruber merged commit 3287750 into main Feb 23, 2023
@juliangruber juliangruber deleted the add/docs-elevator-pitch branch February 23, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants