MichaelPachec0 / nodejs-project-night

Project night for May 27 2022
MIT License
5 stars 12 forks source link

Initial working site implementation. #60

Closed MichaelPachec0 closed 1 year ago

MichaelPachec0 commented 2 years ago

This PR will merge into the master branch a working implementation of the site. This is still a WIP as i have not committed the whole site and (The code has been committed already) There is still the question of what to do when the user gets a full score. (will be implemented in another PR). For the moment the following things are functional:

MichaelPachec0 commented 2 years ago

As of now this is all the work i have done. The last item still needs to be done for which i will try to get to today.

EDIT: In the future i might change a portion of the api to use a POST request (for example checking if the user should get a point)

EDIT 2: I will probably limit the pr to this and create the another that handles the winning condition in another. This already has a ton of changes that will make it difficult to review.

MichaelPachec0 commented 2 years ago

LGTM, one thing, however, from what I recall the API returns an ID for each pokemon card. in the form of an array of objects, each object containing data on the card, and one of those properties being an ID.

The only problem is the client does not retain that info. I could have made that change (keeping the id) either keeping in JavaScript or putting it as a element attribute, but as you can imagine it would have created more changes (and more code commits). Id be more than happy to create another issue (or have anyone else create an issue tracking this) but i would want to keep that seperate to this PR.