Skip to content
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.

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

fix: Set all rich object parameter values to string #12021

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

provokateurin
Copy link
Member

☑️ Resolves

Replaces #11991.
Only strings are allowed as values.

🛠️ API Checklist

🏁 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

@provokateurin
Copy link
Member Author

/backport to stable29

@provokateurin
Copy link
Member Author

/backport to stable28

Signed-off-by: provokateurin <kate@provokateurin.de>
Copy link
Contributor

@SystemKeeper SystemKeeper left a comment

Choose a reason for hiding this comment

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

Works fine on iOS

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

PS: requires review from all requested persons
So blocking until everyone else approved

…nents

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

Adjusted frontend component to match new params. Nothing else is blocking

PS: backport to 27 might end up with conflicts, will handle it

@nickvergessen nickvergessen added feature: chat 💬 Chat and system messages feature: api 🛠️ OCS API for conversations, chats and participants labels Apr 8, 2024
@nickvergessen nickvergessen self-requested a review April 10, 2024 12:20
@nextcloud nextcloud deleted a comment from provokateurin Apr 10, 2024
@nickvergessen nickvergessen merged commit aca8fa1 into main Apr 10, 2024
65 checks passed
@nickvergessen nickvergessen deleted the fix/rich-object-non-string-values branch April 10, 2024 12:27
@mahibi
Copy link

mahibi commented Apr 10, 2024

all fine for android talk as well, @nickvergessen

Antreesy added a commit that referenced this pull request Apr 18, 2024
- in #12021 type of server response was changed to strings, forget to cover tests
- Element.scrollTo is not implemented in JSDOM, need to mock it to avoid errors in Jest

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
backportbot bot pushed a commit that referenced this pull request Apr 18, 2024
- in #12021 type of server response was changed to strings, forget to cover tests
- Element.scrollTo is not implemented in JSDOM, need to mock it to avoid errors in Jest

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug feature: api 🛠️ OCS API for conversations, chats and participants feature: chat 💬 Chat and system messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants