coderoom #11

Merged
clement.freville2 merged 10 commits from coderoom into master 11 months ago
There is no content yet.
bastien.ollier added 2 commits 11 months ago
continuous-integration/drone/push Build is passing Details
57ee11bfea
add button créer une salle
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
819c0e2c74
add editor live
clement.freville2 requested review from clement.freville2 11 months ago
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
4d51e15ab7
gitignore
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/pr Build is passing Details
continuous-integration/drone/push Build is passing Details
ffd2afbc41
merge editor and editor-live
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
7d0fc24b8c
npm start fmt
clement.freville2 requested changes 11 months ago
(click)="onRunButtonClicked()"
[disabled]="isLoaded">
Créer une salle
</button>

Hide this button when the editor is already collaborative. Change the click handler to ask the server for a new room id and redirects the user to that route.

Hide this button when the editor is already collaborative. Change the click handler to ask the server for a new room id and redirects the user to that route.
clement.freville2 marked this conversation as resolved
color: white;
padding: 12px 16px;
font-size: 16px;
width: 100px;
-  width: 100px;
```diff - width: 100px; ```
clement.freville2 marked this conversation as resolved
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/pr Build is passing Details
continuous-integration/drone/push Build is passing Details
89e1fff2c1
add multiroom
bastien.ollier requested review from clement.freville2 11 months ago
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/pr Build is passing Details
continuous-integration/drone/push Build is passing Details
214e3c265d
add multiroom
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
e937c4544a
add multiroom
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
52dfc0e75e
retour merge request
clement.freville2 requested changes 11 months ago
async onCreateRoomButtonClicked() {
const idRoom = await this.backendService.createRoom();
this.router.navigate([`./editor-live/${idRoom}`]);
-    this.router.navigate([`./editor-live/${idRoom}`]);
+    await this.router.navigate([`./editor-live/${idRoom}`]);
```diff - this.router.navigate([`./editor-live/${idRoom}`]); + await this.router.navigate([`./editor-live/${idRoom}`]); ```
clement.freville2 marked this conversation as resolved
constructor() {}
async createRoom() {
const reponse = await fetch(this.apiUrl + '/live');
-    const reponse = await fetch(this.apiUrl + '/live');
+    const reponse = await fetch(this.apiUrl + '/live', { method: 'POST' });
```diff - const reponse = await fetch(this.apiUrl + '/live'); + const reponse = await fetch(this.apiUrl + '/live', { method: 'POST' }); ```
clement.freville2 marked this conversation as resolved
bastien.ollier added 1 commit 11 months ago
continuous-integration/drone/pr Build is passing Details
continuous-integration/drone/push Build is passing Details
09b0e1b2ee
retour merge request
clement.freville2 approved these changes 11 months ago
clement.freville2 left a comment
Owner

Great work!

Great work!
clement.freville2 merged commit 7dfd83b60a into master 11 months ago
clement.freville2 deleted branch coderoom 11 months ago

Reviewers

clement.freville2 approved these changes 11 months ago
continuous-integration/drone/pr Build is passing
continuous-integration/drone/push Build is passing
The pull request has been merged as 7dfd83b60a.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sandkasten/sandkasten-web#11
Loading…
There is no content yet.