Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Integrated chat broken #59

Closed
2 tasks done
Planeshifter opened this issue Feb 22, 2024 · 4 comments · Fixed by #67
Closed
2 tasks done

[Bug]: Integrated chat broken #59

Planeshifter opened this issue Feb 22, 2024 · 4 comments · Fixed by #67
Labels

Comments

@Planeshifter
Copy link
Member

Description

The chat functionality on the website is broken. When opening the chat via the button in the bottom-right corner, one is greeted with:

image

This is likely due to the Gitter Sidecar project being abandoned and the migration to Element.

Related Issues

Related is #33, when the functionality was added.

Questions

No.

Demo

No response

Reproduction

No response

Expected Results

No response

Actual Results

No response

Environments

Firefox, Chrome, Safari, Microsoft Edge, Internet Explorer, Other Browser

Browser Version

No response

Node.js / npm Version

No response

Platform

No response

Checklist

  • Read and understood the Code of Conduct.
  • Searched for existing issues and pull requests.
@lovelindhoni
Copy link
Contributor

@Planeshifter Just as you said, chat's broken since Gitter moved to Element, which in turn led to the abandonment of sidecar's development. I have gone through some possibilities to fix this, but unfortunately, I couldn't find a viable alternative to sidecar.

There are some issues which need to be kept an eye on:
matrix-org/matrix-viewer#157
matrix-org/matrix-hookshot#631
element-hq/roadmap#26

One possibility is to iframe the room, but I don't think that would be a viable option. I would suggest reverting #33 temporarily until a solution is found.

@kgryte
Copy link
Member

kgryte commented Mar 20, 2024

@lovelindhoni I think the main option for now is just to move chat integration altogether.

@lovelindhoni
Copy link
Contributor

lovelindhoni commented Mar 20, 2024

Yeah @kgryte , I tried iframing and it doesn't gone as expected. I also think moving the chat integeration altogether is the best option.

@kgryte
Copy link
Member

kgryte commented Mar 20, 2024

@lovelindhoni If you want to take that on, feel free to submit a PR.

kgryte pushed a commit that referenced this issue Mar 21, 2024
PR-URL: 	#67
Closes: #59
Reviewed-by: Athan Reines <kgryte@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants