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

2024 07 10 zio fullstack webapp #56

Merged

Conversation

cheleb
Copy link

@cheleb cheleb commented Aug 10, 2024

No description provided.

Copy link

netlify bot commented Aug 10, 2024

Deploy Preview for rockthejvmblog ready!

Name Link
🔨 Latest commit aea5a7b
🔍 Latest deploy log https://app.netlify.com/sites/rockthejvmblog/deploys/670060f4501a570008489d88
😎 Deploy Preview https://deploy-preview-56--rockthejvmblog.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 92
Accessibility: 92
Best Practices: 92
SEO: 93
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@daniel-ciocirlan daniel-ciocirlan left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together!
Let me know if you need any help clarifying the comments.

_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
_posts/2024-07-01-zio-fullstack-webapp.md Outdated Show resolved Hide resolved
}
```

In this example Chart.js binding are generated from the TypeScript definition and fully accessible from ScalaJS at compile time, whereas the chart.js will be used at runtime.

Choose a reason for hiding this comment

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

bindings are...

In general, I saw the need for a lot of plural 's' here in the article

This is a very powerful combination, that allows to build a full stack webapp in a type safe manner. And in the mean time we can leverage the JavaScript ecosystem, without compromising type safety, architecture and scalability of the application.


# Roadmap

Choose a reason for hiding this comment

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

Let's cut this Markdown to a sentence. There's no need to add new H1, H2 headers, "I failed" etc. You're doing great, and you want the reader to know that you'll do more. Here's an example:

"The g8 starter project needs an example of JWT token handling, splitting the JavaScript into multiple files, and maybe an example of WASM now that Scala.js has a first full implementation of WasmGC (add link). This is the subject of a future article."

Copy link
Author

Choose a reason for hiding this comment

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

Clearly much more positive 😳

* ScalaJs

* ZIO
* Laminar
* Tapir

* Vite
* ScalablyTyped
@cheleb cheleb force-pushed the 2024-07-01-zio-fullstack-webapp branch 2 times, most recently from e633105 to 10b2f74 Compare September 23, 2024 21:01
@daniel-ciocirlan
Copy link

Thanks for getting back on this one @cheleb! I'll make sure to do another thorough review over the next couple of days.

@cheleb
Copy link
Author

cheleb commented Sep 24, 2024

Please, don't ... I will make another update ASAP

@cheleb
Copy link
Author

cheleb commented Oct 4, 2024

Pushed last version, but I need to check the typo and "ssssssss" :D

Copy link

@daniel-ciocirlan daniel-ciocirlan left a comment

Choose a reason for hiding this comment

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

I'll approve and merge, and fix the nits myself so that we don't delay this article any longer. I'll also shoot a YouTube video to showcase your g8 template!

@daniel-ciocirlan daniel-ciocirlan merged commit 0c92a92 into rockthejvm:master Oct 15, 2024
4 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