Skip to content

fix(filesintegration): Correctly handle optional values for file inte… #16486

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.

Sign up for GitHub

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

Jump to bottom
Merged
nickvergessen merged 1 commit into from
Dec 5, 2025
Merged

fix(filesintegration): Correctly handle optional values for file inte… #16486

nickvergessen merged 1 commit into from
Dec 5, 2025
+2 −1

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Dec 5, 2025

…gration

☑️ Resolves

🛠️ API Checklist

  • Only explanation I have is the file has no owner, which means groupfolders or something
  • It's a bit weird that static analysis did not complain

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen added this to the 🍏 Next Major (33) milestone Dec 5, 2025
@nickvergessen nickvergessen added 3. to review bug feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents labels Dec 5, 2025
@nickvergessen
Copy link
Member Author

nickvergessen commented Dec 5, 2025

/backport to stable32

@nickvergessen
Copy link
Member Author

nickvergessen commented Dec 5, 2025

/backport to stable31

@nickvergessen nickvergessen self-assigned this Dec 5, 2025
@nickvergessen nickvergessen merged commit 485193c into main Dec 5, 2025
81 checks passed
@nickvergessen nickvergessen deleted the bugfix/16477/get-accesslist branch December 5, 2025 21:44
@solracsf
Copy link
Member

solracsf commented Dec 6, 2025

Backport bot seems sleepy 😴

@solracsf
Copy link
Member

solracsf commented Dec 6, 2025

/backport to stable32

@backportbot backportbot bot mentioned this pull request Dec 6, 2025
Merged
@solracsf
Copy link
Member

solracsf commented Dec 6, 2025

/backport to stable31

@backportbot backportbot bot mentioned this pull request Dec 6, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@solracsf solracsf solracsf approved these changes

@miaulalala miaulalala Awaiting requested review from miaulalala

Assignees

@nickvergessen nickvergessen

Labels

3. to review bug feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents

Projects

None yet

Milestone

🍏 Next Major (33)

Development

Successfully merging this pull request may close these issues.

Exception: array_merge(): Argument #2 must be of type array, null given

3 participants

@nickvergessen @solracsf