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

Jd leaderboardservicemix #25

Merged
merged 32 commits into from
Mar 28, 2022
Merged

Jd leaderboardservicemix #25

merged 32 commits into from
Mar 28, 2022

Conversation

jacobwdv
Copy link
Contributor

WIP - leaderboard , client, server, mongodb. Packaged with a docker-compose.yml file to configure interconnectivity.

code for server isn't finished , need to complete communication code for the rover and gameboard.

root and others added 27 commits February 24, 2022 09:55
prototyping idea's for socket comm

adding host=* in server xml so that i can access endpoints running in fyre

adding tests and debugging issues

adding missing constants file

wip

2 way communication, wip

fixing parser

adding message sent to client on 'onOpen' to make client testing easier

wip

typo

removing additional print in test
…se.yml which is the hostname assigned by the docker network
…l be started and networked in a single docker compose call. Tested interconnectivity with leaderboard/mongodb/server
@jacobwdv jacobwdv requested a review from mshah0722 March 25, 2022 14:26
@jacobwdv jacobwdv self-assigned this Mar 25, 2022
jacobwdv and others added 4 commits March 25, 2022 13:01
into JD_leaderboardservicemix

# Conflicts:
#	client/src/App.tsx
#	client/src/components/Layout.tsx
#	client/src/hooks/useTimer.ts
#	client/src/lib/config.ts
#	client/src/pages/PlayPage.tsx
Copy link
Contributor

@mshah0722 mshah0722 left a comment

Choose a reason for hiding this comment

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

LGTM

@jacobwdv jacobwdv merged commit 1c3a2bf into main Mar 28, 2022
@jacobwdv jacobwdv deleted the JD_leaderboardservicemix branch March 28, 2022 14:19
@jimmy1wu jimmy1wu mentioned this pull request Mar 29, 2022
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.

3 participants