Mohit Ruwatia comments

Results 141 comments of


                                            Mohit Ruwatia

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

@lkbhitesh07 PTAL

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

@seanlip PTAL

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

Hey @seanlip I've addressed your [comment](https://github.com/oppia/oppia/pull/20544#discussion_r1658956201), PTAL. Thanks!

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

@seanlip addressed some comments and made the required changes for others. PTAL. Thanks!

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

@seanlip friendly ping

Fix #20399: Added tests for POST, PUT and DELETE and error handling for invalid method using 405 status code.

Hey @seanlip I'll make the changes you requested and try to finish the PR by the 30th. Very sorry for the delay.

[Flake]: Element JSHandle@node took too long to be clickable.

Also occurred on #20483 on this run: https://github.com/oppia/oppia/actions/runs/9585626305/job/26433887822?pr=20483#logs

[Flake]: Element JSHandle@node took too long to be clickable.

Also occurred on #20534 on this run: https://github.com/oppia/oppia/actions/runs/9622874984/job/26545673324?pr=20534#logs

[Flake]: Element JSHandle@node took too long to be clickable.

Also occurred on #20544 on this run: https://github.com/oppia/oppia/actions/runs/9628434511/job/26557692071?pr=20544

[BUG]: Next Chapter Button Overlapped by Footer on Mobile Viewport.

Hi @Sidla this looks great! Great job on demonstrating your changes, just make sure these changes are not making the desktop site behave unexpectedly. Assigning you the issue so you...