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

Demo upgrades #41

Merged
merged 6 commits into from
Dec 15, 2023
Merged

Demo upgrades #41

merged 6 commits into from
Dec 15, 2023

Conversation

mbicknese
Copy link
Contributor

I wanted to start contributing to this project, but I noticed the demo project could use some updates. This PR aims to make it usable again and instantly get new contributors up and running.

The commits are in conventional commits format and can be merged to master without squashing, if so desired.

The styling is opinionated but straightforward.

iPad screenshot

image

Desktop screenshot

image

This is the result of running

```sh
deno run -A -r https://fresh.deno.dev/update .
```
- Ported Fresh components
- Switched to Tailwind, coming from Twind
- Made HTML semantic
- Added some opinionated styling
@mbicknese
Copy link
Contributor Author

Oh! I just noticed the demo works on a very old version of Pentagon. I'll look into that as well.

@skoshx
Copy link
Owner

skoshx commented Dec 15, 2023

@mbicknese Hey, great to hear that you're interested in contributing! Have been super busy lately, so haven't had so much time to put on Pentagon as I would like to!

And thanks a lot for fixing the demo, I do agree it's always nice to have a demo or something that you can start playing around with immediately.

Everything looks good to me at a glance.

There are a few great starting points for contributions, for instance #40 (basically feature parity with Prisma, although keeping mind-ful that we do want to keep the code-base as simple as possible), and #42 (Making Pentagon KV-store agnostic) and #43 to improve the indexing and performance.

@skoshx skoshx merged commit 0342387 into skoshx:main Dec 15, 2023
2 of 3 checks passed
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