Skip to content

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Nov 26, 2022

  • ⚠️ This backport had conflicts and is incomplete

backport of #35361

@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 22.2.11 milestone Nov 26, 2022
@danxuliu danxuliu changed the title [stable22] [stable23] Fix mentions rendering in comment editor [stable22] Fix mentions rendering in comment editor Nov 26, 2022
@danxuliu
Copy link
Member

/compile amend /

NcRichContentEditable needs an index of users to properly display them. This commit adds a caching logic and provides it to NcRichContentEditable.

Signed-off-by: Louis Chemineau <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@danxuliu danxuliu force-pushed the backport/35361/stable22 branch from 911889b to 496b9a3 Compare November 26, 2022 18:42
Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

Built manually on node 16 because compile bot (as well as node CI check) fails on node 18 (node 16 was used in the last successful node CI check on stable22).

@artonge artonge merged commit 8fef74e into stable22 Dec 1, 2022
@artonge artonge deleted the backport/35361/stable22 branch December 1, 2022 17:29
@artonge
Copy link
Contributor

artonge commented Dec 1, 2022

Saperlipopette, wrongly merge before #35463

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants