Skip to content

Conversation

@andersfischernielsen
Copy link
Collaborator

The PR gets the develop branch up to speed:

  • Merged working master into develop
  • Updated all package.jsons using yarn upgrade
  • Made sure all projects still build with upgraded dependencies

@andersfischernielsen andersfischernielsen changed the title Develop Get develop branch working/up to speed Oct 14, 2019
@andersfischernielsen
Copy link
Collaborator Author

The Sass Loader error seems to be fixable by doing this: vuejs/vue-cli#4513

@andersfischernielsen andersfischernielsen changed the title Get develop branch working/up to speed Get branch up to speed/upgraded Oct 14, 2019
@anders-wind
Copy link
Collaborator

There seems to still be a problem with establishing connection between the client and the server. Might be due to updates to sockets?

@andersfischernielsen
Copy link
Collaborator Author

Did we upgrade to Socket.IO 2.0? I can't find anything in the dependabot mess..

@andersfischernielsen
Copy link
Collaborator Author

Apparently we did. It's 1.4.X on master. Getting really sick of this bot TBH

@andersfischernielsen
Copy link
Collaborator Author

andersfischernielsen commented Oct 15, 2019

Could the connection errors be ID-related? See https://socket.io/blog/socket-io-2-0-0/ && socketio/socket.io-client#1058

@andersfischernielsen
Copy link
Collaborator Author

So, it seems that the call at https://github.com/Awia00/Pilgrims/blob/eae3326bf674e51a67f9410f3bdf6dd3ca875409/pilgrims-client/src/store/store.ts#L27 never resolves. I'm unsure what Axios does, but the await never resolves (also not after removing the double await we currently have on develop)

@andersfischernielsen
Copy link
Collaborator Author

andersfischernielsen commented Oct 15, 2019

After more researching, it looks like the mongo DB cannot be reached, that's why the Axiom call waits (tested by running docker run mongo -d and Server+Client using yarn)

@anders-wind
Copy link
Collaborator

Can you try with docker run -d -p 27017:27017 mongo

@anders-wind
Copy link
Collaborator

I think the double await is there for a reason (might be something with await for the reply and awaiting the json deserialization)

@andersfischernielsen
Copy link
Collaborator Author

For some reason the flatMap function is missing whenever Client receives data from Server. This currently breaks the "Start Game" flow. But the connection is fine.

@andersfischernielsen
Copy link
Collaborator Author

Now Client is able to flatMap... Slightly annoying fix, but there's no way to hook into Socket.IO to specify a custom serializer/deserializer AFAIK

@anders-wind
Copy link
Collaborator

Now Client is able to flatMap... Slightly annoying fix, but there's no way to hook into Socket.IO to specify a custom serializer/deserializer AFAIK

Nice! Its not that ugly, but mostly just super weird that it types as if it exists but does not.

@andersfischernielsen
Copy link
Collaborator Author

Yeah, it's super strange. I think the type annotations just override whatever is actually present, which is worrying... 🙁

@andersfischernielsen
Copy link
Collaborator Author

I'm getting some weird "PIXI is missing" errors in Client now, but it builds without errors. Strange. Can you reproduce @Awia00?

@andersfischernielsen
Copy link
Collaborator Author

Woah, dependabot

dependabot-preview bot and others added 25 commits April 6, 2020 04:42
…client/develop/vue/cli-plugin-babel-4.3.0
…client/develop/vue/cli-plugin-unit-jest-4.3.0
…client/develop/vue/cli-plugin-unit-jest-4.3.1
…client/develop/vue/cli-plugin-typescript-4.3.1
…client/develop/vue/cli-plugin-babel-4.3.1
@anders-wind
Copy link
Collaborator

I'm going to merge this into master to try to get it this project moving a bit again :)

@anders-wind anders-wind merged commit 1acac21 into master Apr 12, 2020
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