nextcloud-libraries / nextcloud-libraries/nextcloud-vue

Limit number of server requests for avatars and statuses

Open
#1,618 0 comments 4 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Vue
Stars
246
Forks
99
Avg merge
1d 20h
Merged PRs (30d)
103

Description

Often times an important XHR call is blocked because either avatars or user statuses are being loaded.
This is notable in the Talk app when trying to join a call while the above is still loading.
It happens because browsers limit the number of parallel calls. Not sure if cached calls are counted here.

I suggest we look into using a p-queue to limit the number of XHR requests and possibly image loading requests for avatars and statuses.

A similar approach could be used when loading thumbnails in core and FilePreview in the Talk app... maybe can do once we move those to nextcloud-vue.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the avatar and user-status loading paths involved in the Talk app, then review the suggested p-queue approach for bounding XHR and image requests. The issue names no files or tests; done means important calls such as joining a call are not blocked by unbounded avatar or status loading, while thumbnail work remains separately scoped.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend, performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.