Adds fix to validate if session is still valid on chat. #51

aldompe95 · 2023-05-04T19:51:33Z

Adds validation to verify if session is still valid when using the chat, if the session is not valid redirects to home page that will follow the login process again.

obie · 2023-05-04T20:02:15Z

app/javascript/controllers/chat_controller.js

@@ -29,7 +29,28 @@ export default class extends ApplicationController {
    if (this.element.value.length === 0) {
      return;
    }
-    this.stimulate('ChatReflex#prompt')
+
+    const xhr = new XMLHttpRequest();


When would this fire off?

a better solution is probably to put the check in the ApplicationReflex, similar to how it works in ApplicationController

would be executed after session verification its done.

Yes let me check how to do that

Adds fix to validate if session is still valid on chat.

9983a7c

aldompe95 requested a review from obie May 4, 2023 19:51

obie requested changes May 4, 2023

View reviewed changes

Remove duplicate code

35354f9

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adds fix to validate if session is still valid on chat. #51

Adds fix to validate if session is still valid on chat. #51

aldompe95 commented May 4, 2023

obie May 4, 2023

obie May 4, 2023

aldompe95 May 4, 2023

Adds fix to validate if session is still valid on chat. #51

Are you sure you want to change the base?

Adds fix to validate if session is still valid on chat. #51

Conversation

aldompe95 commented May 4, 2023

obie May 4, 2023

Choose a reason for hiding this comment

obie May 4, 2023

Choose a reason for hiding this comment

aldompe95 May 4, 2023

Choose a reason for hiding this comment