Merge pull request #142 from thecodingmachine/optimize_event_sent

Limiting the number of messages sent while moving

Adding PlayersPositionInterpolator to interpolate/extrapolate players positions

Adding PlayerMovement class to interpolate/extrapolate position along tests (installing Jasmine)

Sending position only every 200ms while moving

Merge pull request #140 from thecodingmachine/fix-animation

Fix animation player

Fix animation player

Merge pull request #138 from thecodingmachine/caching-bundle-webpack

Update strategy caching bundle front js files with webpack

Update strategy caching bundle front js files with webpack

 - Add HtmlWebpackPlugin to use template index html

Merge pull request #135 from thecodingmachine/track_server_load

Tracking nb users connected and server load

Tracking nb users connected and server load

When a user connects or disconnects on the server, we log this (along the current server load).
This is only temporary, in order to assess the number of users a server can safely handle before crashing.

Merge pull request #134 from thecodingmachine/map-coders

Add a coders exit to https://npeguin.github.io/coders-map/map.json

Fix issue on start point

Add a coders exit to https://npeguin.github.io/coders-map/map.json

Merge pull request #133 from thecodingmachine/cleanup_namespace

Trying to trigger delete on correct branch

Trying to trigger delete on correct branch

Merge pull request #132 from thecodingmachine/cleanup_namespace

Fixing Docker again

Fixing Docker again

Merge pull request #131 from thecodingmachine/cleanup_namespace

Fixing Docker reference to image

Fixing Docker

Merge pull request #130 from thecodingmachine/cleanup_namespace

Adding cleanup action on delete

Adding cleanup action on delete

Merge pull request #129 from thecodingmachine/rework_login_scene

Adding a new SelectCharacterScene

Adding localStorage for selected user

Storing user name in local storage

Fix lint

Fixing import

Adding communication between LoginScene and SelectCharacterScene

Fixing selection on keyboard after selection using mouse

Adding a new SelectCharacterScene

Taking what was done in the LoginScene regarding character selection and putting it in its own scene.
Also, making character selection possible via the keyboard.

Merge pull request #127 from thecodingmachine/remove_dead_code

Removing dead code from previous messaging system

Commenting out the notion of status in the GameManager

It seems to be useless

Also removing dead class

Removing dead code from previous messaging system

Remvoing messages that were no more used in the new messaging system (the code used to handle the message that sent the position of all users on the front side)

Merge pull request #126 from thecodingmachine/typos

Fixing typos.

Fixing typos.
Replaced connexion with connection and LogincScene with LoginScene

Merge pull request #124 from moufmouf/rename_maps_endpoint

Renaming /maps to /start-map

Renaming /maps to /start-map

Closes #90

Improving doc

Merge pull request #123 from thecodingmachine/map_instances

Adding a notion of instances per map

Adding a notion of instances per mapAdding a notion of instances to room

The URL signature becomes:

https://workadventu.re/_/[instance]/[path_to_map.json]

This allows us to create many instances of the same map (and therefore to create several different worlds for different people)

An exit on a map can target another "instance" by passing the "exitInstance" property.

Merge pull request #120 from thecodingmachine/refactor_messages

Refactored and optimized messages

Merge pull request #122 from thecodingmachine/reconnecting_scene

Adding a special scene to wait for server reconnection

Merge pull request #121 from thecodingmachine/fix_mediatream_manage

Fix mediatream manage and connection server down

Fix CI

Fix test socket

Prettifying lost connection screen

Adding a special scene to wait for server reconnection

Use `userId` generated by back end in all message

Fix mediam stream manage and server back down

Changing the "Point" notion to add a notion of "moving" in addition to the notion of direction.

Also, refactoring JOIN_ROOM event to add complete position.

Removing logs

Removing dead code

Refactored and optimized messages

Now, when a user moves, only his/her position is sent back to the other users. The position of all users is not sent each time.

The messages sent to the browser are now:

- the list of all users as a return to the join_room event (you can send responses to events in socket.io)
- a "join_room" event sent when a new user joins the room
- a "user_moved" event when a user moved
- a "user_left" event when a user left the room

The GameScene tracks all these events and reacts accordingly.

Also, I made a number of refactoring in the classes and removed the GameSceneInterface that was useless (it was implemented by the LogincScene for no reason at all)

Merge branch 'master' of github.com:thecodingmachine/workadventure into remove_userid

Refactoring searchClientById

searchClientById was scanning through all open sockets to find the right one (which is inefficient if we have many).
Instead, I created a new Map that directly maps ids to sockets.
Furthermore, this solves a long-standing issue (when a socket is disconnected, we cannot find it anymore in the sockets list but it is still available in the disconnect callback from the map)
As a result, we should not have any remaining circles any more.

Merge pull request #118 from thecodingmachine/remove_userid

Refactoring messages

Stricter compiler

Code cleaning

Cleaning up tests

Removing broken import

Removing the Message class and merging it with MessageUserPosition (since it is only ever used it in MessageUserPosition)

Taking advantage of the TypeScript Constructor Assignment too to reduce the amount of code!

Getting rid of roomId in Message class (this is not needed since all messages sent are for the room we are currently in)

When sharing user position, only position is sent now!

JoinRoom now ONLY sends the roomId.

Fixing test

Sending player details (name + character selected) on connection

More code removal

More code removal

Refactoring messages

Socket.io can stringify JSON messages itself, so there is no need to pass a string to "emit". You can pass a serializable object!

This commit removes all the useless toJson() methods, JSON.serialize and JSON.parse!

Woot!

Merge pull request #116 from thecodingmachine/remove_userid

Completely getting rid of "userid"

Completely getting rid of "userid"

Previously, userid was generated by the "/login" route and passed along.
This commit completely removes the uuid "userid" (and disables the LoginController too and any Jwt check).

"userid" is replaced by the "socket id" of the connection.
So a user is now identified using a socket id, which is unique for a given connection.

Merge pull request #113 from thecodingmachine/display_circle_on_startup

Displaying circle on join

Merge pull request #115 from thecodingmachine/update-webrtc

Update media webrtc

Removing log trace

Minor update floor0 and floor1

Fix border view

Media webrtcA

 - Update peerConnexion manage
 - Add muted microphone logo
 - Add icon user
 - Sound when user enter in room webrtc

Displaying circle on join

So far, someone joining a map would not see the circles of groups already formed until someone moves in the group (because the "circle_moved_or_updated" event was not fired when someone arrives)

This commit fixes this behaviour. Someone entering a room will receive an event for each and every group currently formed.

Minor changes map floor 0

Merge pull request #111 from thecodingmachine/new-floor0

New map floor0

Merge pull request #112 from thecodingmachine/debug/improve_message

More detailed error log to debug disconnect issue

Fixing CD

More detailed error log to debug disconnect issue

New map floor0

Merge pull request #109 from thecodingmachine/Auto-reconnect

Auto-reconnect

Merge pull request #108 from thecodingmachine/bugFixes

fixed a game crash on firefox on http

Auto-reconnect

fixed a game crash on firefox on http

Fixing domain name

Merge pull request #107 from thecodingmachine/domain_name

Deploying master on new workadventu.re domain

Not adding comment on push (only pull_request)

Deploying master on new workadventu.re domain

... but still deploying in the test cluster.

Merge pull request #99 from thecodingmachine/map_history

Adding support for history API

Merge pull request #106 from thecodingmachine/up_to_4

Removing limitation to allow up to 4 players in a group

Removing limitation to allow up to 4 players in a group

Fixing htaccess redirect

Redirecting to the correct map after login screen based on URL provided

Adding history support

Arriving on a new map now changes the URL.
All URLs starting with _ are automatically redirected to index.html by the web server

Refactoring/centralizing map loading in GameManager

Merge pull request #103 from thecodingmachine/catching_errors

Catching errors in socket callbacks