-
Notifications
You must be signed in to change notification settings - Fork 529
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
Discussion: App submission form #366
Comments
@autonome , please @ anybody you think might want to intervene 😊 |
@andrew, you might be the most relevant person to give some feedback on this one. |
@Ricardo-Silva91 both options seem to have some significant drawbacks, I'd suggest sticking closer to GitHub reviews to avoid extra complexity and dependencies on extra systems. |
I don't think we have a need for "friendly" (non-GH) means of approving requests - but we would benefit from a way of submitting them that doesn't require the submitter to make a code PR (aka a web form). @autonome, can you confirm? Am I missing something here? Thanks! |
I defer to @jessicaschilling. If we need non-GH approval, we can layer on later. |
Thinking that option 2 above is the best way to go. AFAIK the purpose of this exercise is to give folks who aren't so GitHub-savvy (or who just don't want a GH account - say, a project's PR person) the ability to add a project to Awesome IPFS. Anyone on the core team end who's likely to be evaluating and approving a submission will already be pretty heavily into our GitHub ecosystem, so I don't see a need for Forestry-only editors for this site. That said - generating a staging link for the result of a Forestry-submitted PR would be extremely useful. That way, an approver doesn't need to bring down the PR and build locally in order to make sure it looks OK visually. But that can just be part of the overall CI for the site. Sound good to everyone? |
Quick note for @Ricardo-Silva91: Please note that this repo's README.md also includes a list of all the apps/articles/etc that appear on the Awesome IFPS website. Please make sure that your form workflow accounts for updating that list, too. 😊 |
Closing this in an effort to reboot this list while keeping the maintainer overhead as low as possible while still making sure the list is awesome! See #478 for more info |
Hello! I'm opening this issue so we can discuss the possible addition of an in-site form for people to add their app to the grid.
I'll open another issue to discuss the contents of the form (#367), this thread is for discussing the following approach (and other new approaches as they appear 😌).
When a user submits a new app through the form, it would invoke a lambda function (perhaps in cloudflare), that would create a new commit with the form information.
Now, this commit can go one of two ways:
false
. The reason for this is so that forestry.io reviewers can see preview and edit/fix it easier, before setting 'approve' totrue
or just deleting it. Still, committing directly to master could lead to a lot of problems 😅 (and with many submissions, de forestry environment can also become hard to navigate 🤔).false
'approved' flag. Forestry editors can look at the content itself later, but this seems like a hassle 😅)Once again all new ideas are welcome 🙏
The text was updated successfully, but these errors were encountered: