Skip to content

Conversation

Lukas742
Copy link
Collaborator

@Lukas742 Lukas742 commented Jun 5, 2025

When implementing the UXC Integration example in our wrapper, I found a few issues, that don't break the example but can cause problems when implemented elsewhere. This PR fixes that.

Additionally, I noticed the following points:

  • The content padding of the different UserSettingsViews seems inconsistent
  • Why are the search scopes filtered, shouldn't this work out of the box or am I missing something here? (In the wcr example I omitted the filter logic and it still seems to function as intended) I missed something 😄 you can disregard this point

@Lukas742 Lukas742 requested a review from petyabegovska June 5, 2025 10:00
@nnaydenow
Copy link
Contributor

Hi @dobrinyonkov,

I saw that you committed to these files recently. Could you please take a look?

@nnaydenow nnaydenow requested a review from dobrinyonkov June 6, 2025 06:59
Copy link
Contributor

@dobrinyonkov dobrinyonkov left a comment

Choose a reason for hiding this comment

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

Thanks!

@ilhan007 ilhan007 merged commit 1841dca into main Jun 10, 2025
26 of 28 checks passed
@ilhan007 ilhan007 deleted the docs/uxc-integration-fix branch June 10, 2025 06:46
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.

5 participants