Skip to content

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