Welcome to Software Development on Codidact!
Will you help us build our independent community of developers helping developers? We're small and trying to grow. We welcome questions about all aspects of software development, from design to code to QA and more. Got questions? Got answers? Got code you'd like someone to review? Please join us.
Activity for fausty
Type | On... | Excerpt | Status | Date |
---|---|---|---|---|
Comment | Post #288062 |
> Separately from the review, since you mentioned wanting to avoid paying for static sites
I mentioned this mostly to avoid doing anything that would create the need for server-side rendering. I'm incorporating this into my larger project of turning my blog into a common-place book. I thought th... (more) |
— | over 1 year ago |
Comment | Post #288062 |
> Is it your intention to reduce the text size rather than wrapping the text onto the next line? It's worth narrowing the window as far as it will go to see how small the text becomes, and whether it is still readable.
My intention here is to get the text looking good and consistent across devices... (more) |
— | over 1 year ago |
Comment | Post #288062 |
> It's worth considering updating the quoteText field with a message about the failure for non-technical users, in addition to the full error message in the console.
I haven't added in error handling yet. I'll make this a task.
> The previous 2 functions are prefixed export. Is this because t... (more) |
— | over 1 year ago |
Comment | Post #288062 |
First off, thank you! This is a really thoughtful and helpful review. I think. My plan is to do more work with JavaScript because it's widely used and runs just about everywhere. I've tried TypeScript and do enjoy that more, but I wanted to get at least one thing deployed and running with just HTML/C... (more) |
— | over 1 year ago |
Edit | Post #288054 |
Post edited: |
— | over 1 year ago |
Edit | Post #288054 | Initial revision | — | over 1 year ago |
Answer | — |
A: Ignore NPM Modules Digital Ocean App Platform I solved this last night and meant to come back and update, the problem is totally mine. Jest requires a package-lock.json file in GitHub actions to run. DigitalOcean App platform checks for a Dockerfile if you have one, or package.json/ package-lock.json, it'll follow those rules and the app... (more) |
— | over 1 year ago |
Comment | Post #288042 |
Oh, I solved this last night and meant to come back here and post. It's related to the Digital Ocean App Platform and Github Actions. The issue is: I didn't want App Platform to do anything with Node, since my code is unpackaged html/css/js and doesn't need it to run. I'll post the solution. (more) |
— | over 1 year ago |
Edit | Post #288042 | Initial revision | — | over 1 year ago |
Question | — |
Ignore NPM Modules Digital Ocean App Platform There may be a better way to manage website's with JavaScript deployments. I added jest to my repo simply to execute tests when I make PRs as part of a GitHub action. This requires a package-lock.json to work. Unfortunately, the presence of this file will automatically kick off an `npm install` i... (more) |
— | over 1 year ago |
Edit | Post #288041 | Initial revision | — | over 1 year ago |
Question | — |
Vanilla JS Functions Review I wanted to create a simple static website that would use JS to randomly rotate through a quotes.json file I maintain. My JavaScript experience is more limited as it's not what I do at work, so what I'm seeking is good JS practices and writing in a way that is "idiomatic" to the JS community. My ... (more) |
— | over 1 year ago |