Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Add: Home, What, About and Self Hosting info pages with routing (DRAFT) #6

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

Conversation

amsalmeron
Copy link
Collaborator

@amsalmeron amsalmeron commented Aug 3, 2023

Issue #5 : create static pages What, About and Self Hosting. Also added a Home page at the root. Set up routing in +page.svelete. Components should probably be in another directory, and proof reading is appreciated.

This is a DRAFT. CSS non existent.

Vercel: https://nostrkit.vercel.app/

@eriknewland
Copy link

this is great, thanks! Components architecture for svelte kit should look like:

Screen Shot 2023-08-03 at 5 49 33 PM

using the 'lib' directory makes imports really easy:

Screen Shot 2023-08-03 at 5 50 34 PM

we will probably need @bitkarrot to help clarify some of the self hosting stuff. I should have a PR with the base nostr-kit logic pushed tonight/tomorrow

@bitkarrot
Copy link
Member

OK, @amsalmeron @eriknewland gimmie a day or two to catch up here. Am way behind. if you don't get a response in a day or so, ping me again.

<p>6. Once deployment is complete, your project will be live on Heroku and you will be able to access it using the URL provided</p>
</ol>
</ol>
<h3>Congratualtions</h3>
Copy link
Member

Choose a reason for hiding this comment

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

Some typos in the 'Congratulations' section on https://nostrkit.vercel.app/selfhosting, please spell check and proof.

@bitkarrot
Copy link
Member

@eriknewland and I will discuss UX/UI in the next day or so

Copy link
Member

@bitkarrot bitkarrot left a comment

Choose a reason for hiding this comment

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

please see review notes regarding typos.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants