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

initial theme #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

initial theme #12

wants to merge 2 commits into from

Conversation

publu
Copy link

@publu publu commented Feb 14, 2020

Pull Request

Contribution type

  • [ X] New template contribution
  • Existing template update
  • Issue fix
  • README.md update

Contribution Description

new template added

[ X] I would like my template to be added to the Unstoppable Domains' website builder

Unstoppable Domain Name:

gonna be contacting [email protected] now
If you don't have a domain yet contact [email protected] for a free domain.

@matiasunstoppable
Copy link

Hey,

Matias from Unstoppable Domains here.

Thanks for submitting a template!

We will be reviewing all pull requests this week.

Best,

Copy link
Contributor

@donald-stolz donald-stolz left a comment

Choose a reason for hiding this comment

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

@publu Thanks for your submission. Here are some requests for updates:

  • Make sure your site is mobile responsive
  • Move js files into an assets folder
  • Add an action to your form. Also note that the action should not depend on any centralized APIs

@publu
Copy link
Author

publu commented Feb 20, 2020

Does that mean the submission is accepted and you're requesting this to finalize merging?

The last request is a Catch 22. Either we set up a "decentralized" system to collect emails or we don't. I don't see how that's a reasonable offer unless you prefer removing it completely.

@donald-stolz
Copy link
Contributor

You should checkout 3Box they have plugin tools for decentralized Messaging and Storage that could possibly be used.

We are not currently planning on accepting this submission as part of the initial 10 for the bounty because it is too simple. Feel free to leave the pull request open if you would like to make updates or have it re-reviewed after the bounty

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.

3 participants