fix: Use timezone.utc instead of UTC #6
Conversation
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
| cl.user_session.set("project_id", project_id) | ||
|
|
||
| # get lang_code from the copilot and store it in the session | ||
| lang_code = await self.call_copilot("getLangCode", default="en") |
There was a problem hiding this comment.
this returned a None object at the start of chat or something like that, so that lang_code was None
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
| if store.has_history(user.identifier, project_id): | ||
| content = getattr(config, f"CONTINUATION_{lang_code.upper()}", "") | ||
| await cl.Message(content=content).send() | ||
| history = store.get_history(user.identifier, project_id) |
There was a problem hiding this comment.
I've added here the chat history to the messages below this continuation message. Added it as a feature here but would make sense right?
There was a problem hiding this comment.
I thought about this, I am not sure. I need to check how it looks.
| subject: subjectInput.value, | ||
| message: messageInput.value | ||
| } | ||
| $(submitButton).off('click').on('click', async () => { |
There was a problem hiding this comment.
the submit button for send email was hanging(didn't do anything) for me at some point.
jochenklar
left a comment
jochenklar
left a comment
There was a problem hiding this comment.
Maybe we talk about the changes later this week, I also implemented the other things we discussed, so this needs a rebase.
|
|
||
| MIDDLEWARE.append('rdmo_chatbot.plugin.middleware.ChatbotMiddleware') | ||
|
|
||
| # Address of the Chainlit server serving the copilot widget. Provide an |
There was a problem hiding this comment.
I don't think that will work since the cookie with chatbot_token will hot work.
|
|
||
| try: | ||
| result = await cl.CopilotFunction(name=name, args=kwargs).acall() | ||
| except Exception: |
There was a problem hiding this comment.
This is not a good pattern, was there a specific reason to add this? I am ok with the args -> kwargs, although now it is confusing in a different way.
| return response_message | ||
|
|
||
| async def send_history(self, history, user): | ||
| assistant_name = getattr(config, "ASSISTANT_NAME", "Assistant") |
There was a problem hiding this comment.
just use config.ASSISTANT_NAME there should be a default.
| if store.has_history(user.identifier, project_id): | ||
| content = getattr(config, f"CONTINUATION_{lang_code.upper()}", "") | ||
| await cl.Message(content=content).send() | ||
| history = store.get_history(user.identifier, project_id) |
There was a problem hiding this comment.
I thought about this, I am not sure. I need to check how it looks.
| height: 2em; | ||
| } | ||
|
|
||
| .sr-only { |
| return string.length > maxLength ? string.slice(0, maxLength) + '…' : string; | ||
| } | ||
|
|
||
| const getCookie = (name) => { |
There was a problem hiding this comment.
This is not needed Cookies is in the window object now.
|
|
||
| $(contactModal).modal('hide') | ||
| $(contactModal).modal('hide') | ||
| } catch (error) { |
There was a problem hiding this comment.
Is this really needed or is this just ChatGPT boilerplate?
| const shadow = copilot.shadowRoot | ||
| const observedShadows = new WeakSet() | ||
|
|
||
| const ensureDialogAccessibility = ({ |
There was a problem hiding this comment.
The rest of the file needs refactoring or at least a reason why it is needed. This is just AI code.
| <script src="{{ settings.CHATBOT_URL }}/copilot/index.js"></script> | ||
| <script src="{% static 'chatbot/js/copilot.js' %}"></script> | ||
| <script> | ||
| const configuredChatbotUrl = "{{ settings.CHATBOT_URL }}"; |
There was a problem hiding this comment.
I don't think this is needed.
There was a problem hiding this comment.
All but this change should be a different PR 🥴.
I got an error
cannot import name 'UTC' from 'datetimewith py310