mui/tech-challenge-full-stack

Refresh challenge

Opened this issue ยท 4 comments

We could:

  • Upgrade React to v18 #4

"react": "^17.0.2",

  • Update Next.js to the pages directory

export default class MyDocument extends Document {

Janpot commented

Honestly, I'm not sure we should push candidates even deeper into Next.js's paradigms. It's so opinionated and aimed at the static website use-case. It has these strong constraints on routing, it doesn't support API routes, it's all vendor specific and non-standardized. I'd like to gauge candidates capability to understand the basics of React, and the basic anatomy of web servers. If anything, I think for the future of this challenge we could rather aim towards getting rid of Next.js and offer something more bare bones.

Opening #4 already for the dependencies.

On

Screenshot 2023-10-02 at 17 22 39

this is something that has already surfaced in the past with @samuelsycamore, @danilo-leal, and @mj12albert. I think there is already a general consensus on this, I had the same pain. I have added the point to the TODO list in mui/material-ui#34905 (in cases there is no existing issues about it already)

Honestly, I'm not sure we should push candidates even deeper into Next.js's paradigms. It's so opinionated and aimed at the static website use-case. It has these strong constraints on routing, it doesn't support API routes, it's all vendor specific and non-standardized. I'd like to gauge candidates capability to understand the basics of React, and the basic anatomy of web servers. If anything, I think for the future of this challenge we could rather aim towards getting rid of Next.js and offer something more bare bones.

@Janpot Should we go down the Vite + express API route then?

Janpot commented

๐Ÿ™‚ I typed that comment because I got frustrated with the example. I removed it again after I started playing around with app dir and this challenge and it didn't feel like a good fit. But fair enough, should have signaled this ๐Ÿ‘

Should we go down the Vite + express API route then?

I think that would be a good stack for the challenge. The only thing that holds me back from it is vitejs/vite#12423. It would be a pity doing a coding challenge and then have this bug eat all your time.