coderoom #11

Merged
clement.freville2 merged 10 commits from coderoom into master 1 year ago
Owner
There is no content yet.
bastien.ollier added 2 commits 1 year 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 1 year ago
bastien.ollier added 1 commit 1 year ago
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
4d51e15ab7
gitignore
bastien.ollier added 1 commit 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago
bastien.ollier added 1 commit 1 year 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 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago
clement.freville2 left a comment
Owner

Great work!

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

Reviewers

clement.freville2 approved these changes 1 year 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.