Skip to content

[NoQA] Update the comments for getTooltipShiftX#7593

Merged
yuwenmemon merged 1 commit into
mainfrom
vit-updateTooltipShiftComment
Feb 7, 2022
Merged

[NoQA] Update the comments for getTooltipShiftX#7593
yuwenmemon merged 1 commit into
mainfrom
vit-updateTooltipShiftComment

Conversation

@mountiny

@mountiny mountiny commented Feb 6, 2022

Copy link
Copy Markdown
Contributor

Details

Just a simple change to comments to fix some typos and make it easier to read and understand.

Fixed Issues

N/A

Tests

  • Verify that no errors appear in the JS console

QA Steps

  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

@mountiny mountiny self-assigned this Feb 6, 2022
@mountiny mountiny requested a review from a team as a code owner February 6, 2022 19:22
@MelvinBot MelvinBot requested review from yuwenmemon and removed request for a team February 6, 2022 19:23

@yuwenmemon yuwenmemon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Amazing. 😄

@yuwenmemon yuwenmemon merged commit 0335591 into main Feb 7, 2022
@yuwenmemon yuwenmemon deleted the vit-updateTooltipShiftComment branch February 7, 2022 20:06
@OSBotify

OSBotify commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

OSBotify commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @yuwenmemon in version: 1.1.38-0 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @Julesssss in version: 1.1.38-3 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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