Convert some capital letters to lowercase and capitalize months and days in Spanish. - #16502
Convert some capital letters to lowercase and capitalize months and days in Spanish.#16502ayazhussain79 wants to merge 2 commits into
Conversation
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
@Julesssss Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
|
Sir most of the changes are on en.js file, so it will applied on all platform I'm facing some issues on emulators sorry for that. you can verify them on any platform |
|
Hi @ayazhussain79, can you please add screenshots/videos for all the platforms you can? And list those you cannot so we can fill in the remaining. Thanks. |
|
@Julesssss Sir, I have tested them on web chrome but not able to test them on other platforms, you can verify them it will be applied on all platforms |
Sorry, but that's not what we expect from a contributor. Please build and test each platform, even if it seems impossible to 'break'. This is a standard requirement for our external issues. There are cases where contributors have issues with a single platform, and in those cases we are happy to help, but not to build the remaining 4 platforms. Thanks. |
|
Also, I see a couple of failures in the CI flows.
|
Julesssss
left a comment
There was a problem hiding this comment.
Left a note with the required changes
|
@Julesssss Ok working on it |
|
@Julesssss I have uploaded other platform videos/SS except Mobile Web - Safari and I have also used GPG keys on my github to verified commit, do I need to create a new branch and pull request? |
|
@Julesssss I have created new PR with signed commits #16593 (comment) |
|
Yep, Thanks! |
Details
We want to convert some capital characters to lowercase and capitalization for months and days in Spanish.
Fixed Issues
#16403
PROPOSAL: #16403 (comment)
Tests
Offline tests
Same as above
QA Steps
Same as above
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)src/languages/*files and using the translation methodWaiting for Copylabel for a copy review on the original GH to get the correct copy.STYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)/** comment above it */thisproperly so there are no scoping issues (i.e. foronClick={this.submit}the methodthis.submitshould be bound tothisin the constructor)thisare necessary to be bound (i.e. avoidthis.submit = this.submit.bind(this);ifthis.submitis never passed to a component event handler likeonClick)StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)Avataris modified, I verified thatAvataris working as expected in all cases)ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Web
2023-03-25-05-02-23_IjBKQjEb.mp4
Mobile Web - Chrome
Uploading Screen Recording 2023

Screen.Recording.2023-03-27.at.10.49.36.PM.mov
Mobile Web - Safari
Desktop
Screen.Recording.2023-03-27.at.10.40.34.PM.mov
iOS
Screen.Recording.2023-03-27.at.10.46.35.PM.mov
Android
Screen.Recording.2023-03-27.at.11.02.56.PM.mov