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

Catching errors in socket callbacks

Catching errors in socket callbacks to avoid having the server crashing when an error occurs.

Minor update layer position in map floor0 and floor1

Merge pull request #98 from thecodingmachine/map-v1

Add KEN office to the map

Fixing exit points

Merge branch 'master' of github.com:thecodingmachine/workadventure into map-v1

# Conflicts:
# back/src/Assets/Maps/Floor0/floor0.json

Merge pull request #100 from thecodingmachine/fix/css

Minor Fix css style

Minor Fix overflow css style

Add KEN office to the map

Merge pull request #93 from thecodingmachine/fix-recette-grp

Fix recette GRP

Merge pull request #95 from thecodingmachine/lyon_map

 Refactoring "exit" to allow multiple maps

Fix depth error in the first floor

Fix CI lint

Fix position layer in map lyon

Merge pull request #97 from moufmouf/fix_player_name_depth

Putting player names depth at a high depth to have them always visible

Fixing typo

Putting player names depth at a high depth to have them always visible

Refactoring "exit" to allow multiple maps

Adding several layers named "exit" causes issues with Phaser 3.
This PR makes any possible layer being an exit layer (it only depends on the exitUrl property)

Also, fixing start position (it takes into account the layer width now)

use start and exit layers on map

Delete params not used

Fix update world back end and deleting group in front end

Merge pull request #88 from thecodingmachine/feature/map-floor1

Map 1st floor

Rewording exit paragraph

Fix exit tile map of floot1

Fix url to get the next map in GameScene

Add explain to have multi exit layers into the map

Fix error mapping refresh in back & the end of this feature development

Refactor leave and join room

Change to update world by scene and room id

Fix feedback @moufmouf strategy of maps managing by back.

Fix feedback @moufmouf on circle texture

Remove middleware to secure access API.

Correct feedback @moufmouf

Update back/src/Controller/MapController.ts

Co-authored-by: David Négrier <d.negrier@thecodingmachine.com>

Update back/src/Controller/MapController.ts

Co-authored-by: David Négrier <d.negrier@thecodingmachine.com>

Permit to dissociate data by room

 - Update share room id.
 - Join room when a scene is loaded.
 - Add a room in constant variable.

Manage multi scene

 - Create position and check if user is in position to switch in the next scene.
 - When scene is load, we load all scene in the layer of name "exit".
 - Layer "exit" of map.json have a parametter "exitSceneKey" to identify next scene.
 - Add layer "start", the player could start in the scene on the object present in the layer of name "start".

Multi Scene in back end

 - Change maps url to get maps
 - Change GameScene to create scene with file since back end
 - Change LoginScene to upload scene and start game

Add maps in back

 - Add all map json in back
 - Create middleware to check authentification user
 - Create controllers to get map
 - Create access to get all files in folder Assets/Maps

Update name map

 - Change name in game scene and login scene

Add map 1st floor

Rename map to floor 0

Merge pull request #85 from thecodingmachine/fix/webrtc-group

Fix WebRtc group

Fix manage webrtc room id

Fix User depth

 - Update strategy layers depth
 - Add depth to manager layers positions between all users

Merge pull request #83 from thecodingmachine/cleanup_rename_frame

Cleanup: renaming "frame" to "character"

Merge pull request #81 from thecodingmachine/display_groups

Adding the display of a circle around the group

Merge pull request #82 from thecodingmachine/fix/webrtc

Fix webrtc

Refactoring display of circle around group

The circle is now a sprite and not generated on every frame using a giant "graphics" object

Renaming frame in tests

Cleanup: renaming "frame" to "character"

The "frame" variable actually contains a string pointing to the character selected.
It has nothing to do with a frame which is usually a particular image in an animation.

I'm renaming the variable accross the application to avoid confusion.

Add layer override

Personalization of DAM office

Fix lint tsc

Fix shares information to enter into Webrtc room.

- Refactor share position in world class.
- Update selects a client to send information about connecting WebRtc room.

Fix webrtc back

 - Add refuse to join if webrtc room have 4 players.

Adding the display of a circle around the group

This PR adds the display of a circle around groups. This is useful to view where you need to go to speak to someone but also to debug.

Note: implementation is suboptimal, relying on a "graphics" object that is known to be slow. In the future, we need to use a circle as a sprite instead.

Merge pull request #80 from thecodingmachine/display_groups

Fixing "none" animation does not exist log

Fixing "none" animation does not exist log