It's currently possible to authenticate without a correct password

#34 (comment)

• Reduce image size by copying only required files

• Plug in the awesome homepage created by @arimet https://codepen.io/arimet/pen/WNdedzv

When it's not the player's turn we are still displaying an hover effect on cells and a mouse pointer too.

It takes a long time to draw a large board (e.g 19x19).

Clebiez On PR #38
Your gameService is a variable which will mutate between different tests.

It's a bad practice and you should avoid that.

Each it function should be completely independant, obviously you can use "beforeEach" and "afterEach" but you must never be able to grab a variable which is updated from another test.

If you have a test with no call to updateGameState, but you assert something, this assert pass but the feature doesn't work because the board that you expected is wrong.
Also you can't be certified that every tests runs will run your test in the same order.
Prefer setting the game in a beforeEach by calling getBoardStateFromFile every time.

Fix this, either using a beforeEach or by calling getBoardStateFromFile every time

user.ts and user.spec.ts are no longer userful and should be removed from the codebase

• Create a shared volume with the host(don't need to copy the code into the container anymore)
• Start the server in dev mode

Either rework this helper to do something closer to what we need, in order to make the index.hbs file easier to read.

Or simply use css only conditions, like in f8ac184

Original PR comment: #16 (comment)

Complete the Readme file to give instructions about how to launch and debug the cypress tests locally

E.g.: mention this command: yarn cypress:open

The game board is truncated on the right of the screen, no matter the resolution or the zoom.

It is especially visible on small board sizes.

Probably has to do with the calculation of the board width.

The default font color is black so when black wins the winning path isn't visible

When logging in as admin for the first time, RA seems to redirect to the GamesList page but then displays an error.
A refresh is sufficient to make the page work again, and afterwards the whole app works great.
Subsequent log out / log in also work fine...

The issue is reproduced in the production environment only.

To reproduce manually, one can remove the "jwt" key in the browser's local storage, and then refresh the page.

Hereby is a demo of the bug:

Mobile app currently allows to play twice at the same position, resulting in an error since the API won't allow it.

Stacktrace:

Board.tsx:19 Uncaught TypeError: Cannot read properties of undefined (reading 'length')
    at Board (Board.tsx:19:1)
    at renderWithHooks (react-dom.development.js:14985:1)
    at updateFunctionComponent (react-dom.development.js:17356:1)
    at beginWork (react-dom.development.js:19063:1)
    at HTMLUnknownElement.callCallback (react-dom.development.js:3945:1)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:3994:1)
    at invokeGuardedCallback (react-dom.development.js:4056:1)
    at beginWork$1 (react-dom.development.js:23964:1)
    at performUnitOfWork (react-dom.development.js:22779:1)
    at workLoopSync (react-dom.development.js:22707:1)
    at renderRootSync (react-dom.development.js:22670:1)
    at performSyncWorkOnRoot (react-dom.development.js:22293:1)
    at react-dom.development.js:11327:1
    at unstable_runWithPriority (scheduler.development.js:468:1)
    at runWithPriority$1 (react-dom.development.js:11276:1)
    at flushSyncCallbackQueueImpl (react-dom.development.js:11322:1)
    at flushSyncCallbackQueue (react-dom.development.js:11309:1)
    at scheduleUpdateOnFiber (react-dom.development.js:21893:1)
    at dispatchAction (react-dom.development.js:16139:1)

We should

• Make the non-empty cells non-clickable
• Have better handling of API errors

Changes in the rendered contents cause flickering

For now, the AI always chooses the move that is top right of the board, among all moves with same cost.

It would be better to have the AI choose randomly, to bring more variety to the gameplay and make its algorithm less obvious.

Original PR comment: #66 (comment)