Skip to content

fix(files): properly update store on files conversions success - #50645

Merged
skjnldsv merged 2 commits into
masterfrom
fix/refresh-convert-list
Feb 4, 2025
Merged

fix(files): properly update store on files conversions success#50645
skjnldsv merged 2 commits into
masterfrom
fix/refresh-convert-list

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

Now we make sure to fetch the new files and properly update the store

If none of the files converted are in the current folder the user is looking at, no nodes will be fetched, as it will when they will browse back into said folder again! 💪
Nice performance boost :)

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

Comment threadapps/files/src/actions/convertUtils.ts Outdated
Comment threadapps/files/src/actions/convertUtils.ts Outdated

@susnuxsusnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just two comments in how to get rid of esline-disable comments ;)

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv
skjnldsvforce-pushed the fix/refresh-convert-list branch from a5e8e6d to af405f6CompareFebruary 4, 2025 15:41
@skjnldsv
skjnldsv enabled auto-merge February 4, 2025 15:47

@AndyScherzingerAndyScherzinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@skjnldsv
skjnldsv merged commit a2e05ee into masterFeb 4, 2025
@skjnldsv
skjnldsv deleted the fix/refresh-convert-list branch February 4, 2025 16:45
@nextcloud-botnextcloud-bot mentioned this pull request Aug 19, 2025
Sign up for freeto 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

@skjnldsv@AndyScherzinger@susnux