Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(SDK-4030): IncorrectContextUseViolation #659

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

CTLalit
Copy link
Collaborator

@CTLalit CTLalit commented Aug 30, 2024

  • uses version check and fixes windowmanager access from context
  • removes redundant code for pixes which was unused

- uses version check and fixes windowmanager fetch
- removes redundant code for pixes which was unused
- reduces service fetches from context
- reduces version checks
- refactors code to add placeholder class
- returns window result as whole
Copy link
Contributor

@Anush-Shand Anush-Shand left a comment

Choose a reason for hiding this comment

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

Left comments

CTLalit and others added 2 commits October 10, 2024 14:01
- catches exception
- removes decimal adjustment which was done twice.
@CTLalit CTLalit requested a review from Anush-Shand October 10, 2024 12:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants