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

perf(sharing): Use getFirstNodeById() which is more performant #12679

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jul 11, 2024

Backport of PR #12554

As we don't care which node we get for rendering the message

Signed-off-by: Joas Schilling <coding@schilljs.com>
@backportbot backportbot bot added 3. to review feature: api 🛠️ OCS API for conversations, chats and participants feature: upload & shares & voice 📤🎙️ Sharing files into a chat and audio recordings deployed on testing performance 🚀 labels Jul 11, 2024
@nickvergessen nickvergessen changed the title [main] [stable29] perf(sharing): Use getFirstNodeById() which is more performant perf(sharing): Use getFirstNodeById() which is more performant Jul 11, 2024
@nickvergessen nickvergessen added this to the 💙 Next Major (30) milestone Jul 11, 2024
@nickvergessen nickvergessen enabled auto-merge July 11, 2024 18:04
@nickvergessen nickvergessen merged commit 2f395fe into main Jul 11, 2024
63 checks passed
@nickvergessen nickvergessen deleted the backport/12554/main branch July 11, 2024 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review deployed on testing feature: api 🛠️ OCS API for conversations, chats and participants feature: upload & shares & voice 📤🎙️ Sharing files into a chat and audio recordings performance 🚀
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant