Skip to content

Conversation

@dajimenezriv-internxt
Copy link
Contributor

@dajimenezriv-internxt dajimenezriv-internxt commented Jan 9, 2026

What

Use lastId to paginate files when sorting by uuid.

@dajimenezriv-internxt dajimenezriv-internxt self-assigned this Jan 9, 2026
isScanCompleted: boolean;
progressRatio: number;
isAntivirusAvailable: boolean;
showErrorState: boolean;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixing a sonarcloud error and some warnings.

@sg-gs
Copy link
Member

sg-gs commented Jan 9, 2026

Did you benchmark any noticeable improvement on speeds / stability by using this approach @dajimenezriv-internxt ?

@dajimenezriv-internxt
Copy link
Contributor Author

dajimenezriv-internxt commented Jan 9, 2026

@sg-gs Not really, here I have a good internet connection and everything feels pretty fast. Also in the activate@internxt.com account there are only 4k files, so that means just 4 calls. Also, in this case it's the recovery sync that runs every 30s and the performance here is not so relevant. The one more problematic is the request that filters by updatedAt.

dajimenezriv-internxt and others added 5 commits January 9, 2026 16:43
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.21 to 4.17.23.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.21...4.17.23)

---
updated-dependencies:
- dependency-name: lodash
  dependency-version: 4.17.23
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@dajimenezriv-internxt dajimenezriv-internxt changed the base branch from main to dependabot/npm_and_yarn/lodash-4.17.23 January 22, 2026 09:32
Base automatically changed from dependabot/npm_and_yarn/lodash-4.17.23 to main January 22, 2026 09:41
@sonarqubecloud
Copy link

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