Skip to content

Conversation

@Sotatek-DucPhung
Copy link
Collaborator

@Sotatek-DucPhung Sotatek-DucPhung commented Oct 17, 2025

Description

Resolve PR comments for ESLint in core

Checklist before requesting a review

Issue ticket number and link

  • This PR has a valid ticket number or issue: VT20-2117

Testing & Validation

  • This PR has been tested/validated in iOS, Android and browser.
  • Added new unit tests, if relevant.

Design Review

  • In case this PR contains changes to the UI, add some screenshots and/or videos to show the changes on relevant devices.

@Sotatek-DucPhung Sotatek-DucPhung marked this pull request as draft October 17, 2025 08:06
@iFergal
Copy link
Collaborator

iFergal commented Oct 20, 2025

Is this ready for review?

@Sotatek-DucPhung Sotatek-DucPhung marked this pull request as ready for review October 21, 2025 11:51
@Sotatek-DucPhung Sotatek-DucPhung force-pushed the feat/VT20-2117-resolve-PR-comments-for-eslint-in-core branch from 0b37254 to e045043 Compare October 31, 2025 08:34
@Sotatek-DucPhung
Copy link
Collaborator Author

@iFergal please take a look again

@iFergal iFergal changed the title Feat/vt20 2117 resolve pr comments for eslint in core feat(core): resolve pr comments for eslint in core Nov 4, 2025
Copy link
Collaborator

@iFergal iFergal left a comment

Choose a reason for hiding this comment

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

awesome work!

@iFergal iFergal merged commit bd6abcc into develop Nov 5, 2025
3 checks passed
@iFergal iFergal deleted the feat/VT20-2117-resolve-PR-comments-for-eslint-in-core branch November 5, 2025 11:11
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.

3 participants