Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 26, 2024

Backport of #43325
Backport of #44162

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from nfebe and skjnldsv February 26, 2024 09:22
@backportbot backportbot bot added this to the Nextcloud 28.0.3 milestone Feb 26, 2024
@skjnldsv skjnldsv mentioned this pull request Feb 28, 2024
@Altahrim Altahrim mentioned this pull request Mar 21, 2024
9 tasks
@skjnldsv skjnldsv requested review from emoral435 and susnux April 4, 2024 12:28
@skjnldsv

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@skjnldsv
Copy link
Member

skjnldsv commented Apr 4, 2024

Rebasing.... ⌛

@skjnldsv skjnldsv force-pushed the backport/43325/stable28 branch from 83381cd to 59cfded Compare April 4, 2024 16:34
@skjnldsv skjnldsv marked this pull request as ready for review April 4, 2024 16:34
@skjnldsv skjnldsv force-pushed the backport/43325/stable28 branch from 59cfded to e22b67d Compare April 4, 2024 16:40
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 4, 2024
@skjnldsv skjnldsv enabled auto-merge April 4, 2024 16:54
const fileId = this.getFileIdFromPath(path)
const node = this.getNodeFromId(fileId)
const fileId: number | undefined = this.getFileIdFromPath(path)
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not get why hard coding this is needed instead of implicit type from function return?

Copy link
Member

Choose a reason for hiding this comment

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

I have no idea 😅
I don't recall when I added this. Might have had a brain Zapp 🙃

@skjnldsv skjnldsv merged commit 85ebbab into stable28 Apr 4, 2024
@skjnldsv skjnldsv deleted the backport/43325/stable28 branch April 4, 2024 17:41
@Altahrim Altahrim mentioned this pull request Apr 17, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish enhancement feature: files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants