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

Arc/msc3971 #15

Open
wants to merge 1 commit into
base: sc
Choose a base branch
from
Open

Conversation

AndrewRyanChama
Copy link

@AndrewRyanChama AndrewRyanChama commented Mar 11, 2023

Fresh delivery for matrix-org/matrix-spec-proposals#3971

Does the following:

  • sends emoji with the new attribute
  • makes the emoji clickable and links to the correct room

This doesn't implement any advanced UI for after clicking the link, but does send the user to the correct room. A future change will be to show the user a big red button to add the pack.


  • I agree to release my changes under this project's license

@su-ex
Copy link
Member

su-ex commented Mar 11, 2023

Could you try to fix at least the newly introduced linting issues?
I've recently tried to not leave them completely unhandled where possible.

@AndrewRyanChama AndrewRyanChama force-pushed the arc/msc3971 branch 3 times, most recently from 00cc499 to a9b950c Compare March 13, 2023 04:44
@AndrewRyanChama
Copy link
Author

@su-ex lint should be fixed now

@su-ex
Copy link
Member

su-ex commented Mar 15, 2023

Seems there are still some linting issues.
Also please fix the conflicts.
Overall great idea, but I'm somehow concerned with the maintenance effort as this changes a lot. But some of those areas are already changed for custom emojis anyway, so probably not a huge additional burden. 🤔

@AndrewRyanChama AndrewRyanChama force-pushed the arc/msc3971 branch 4 times, most recently from 8c7e179 to 50b10a6 Compare May 19, 2023 06:59
make them clickable

fix typescript error checks
@AndrewRyanChama
Copy link
Author

@su-ex can you take a look again now?

I'm pretty sure I can'tfix the other failed checks without changing a whole bunch of stuff all over

SpiritCroc pushed a commit that referenced this pull request Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants