Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

remove datepicker style workaround #1768

Merged
merged 1 commit into from
Feb 11, 2022
Merged

Conversation

jlf0dev
Copy link
Member

@jlf0dev jlf0dev commented Feb 11, 2022

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

We had a workaround for DatePicker styling that targeted iOS 13.2 and above. This workaround was actually supposed to be targeting 14.0 and above, so was throwing an error on anything between 13.2 and 14.0.

Since we have upgraded to Xamarin 5.0.0.2125, we can remove this as Xamarin has fixed the issue

Code changes

  • src/iOS.Core/Renderers/ExtendedDatePickerRenderer.cs: Remove styling workaround

Testing requirements

DatePicker should continue working correctly. There are examples on the Settings and EditSend screens.

Before you submit

  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@jlf0dev jlf0dev merged commit 59ed76d into master Feb 11, 2022
@jlf0dev jlf0dev deleted the bug/datepicker-style-issue branch February 11, 2022 17:23
fedemkr added a commit that referenced this pull request Feb 11, 2022
* master: (34 commits)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)
  Set background of biometricCell from theme (#1744)
  Cleared About information about push registration, just left it in DEBUG mode (#1742)
  Cleared Console.WriteLine for iOS push notifications issue (#1739)
  Autosync the updated translations (#1734)
  Update FA icons on LockPasswordViewController (#1729)
  ...
fedemkr added a commit that referenced this pull request Feb 14, 2022
* master:
  [Icons] - BUG - Update groupings icon for collections (#1773)
  Autosync the updated translations (#1766)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)

# Conflicts:
#	src/Android/Properties/AndroidManifest.xml
#	src/iOS.Autofill/Info.plist
#	src/iOS.Extension/Info.plist
#	src/iOS.ShareExtension/Info.plist
#	src/iOS/Info.plist
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.

None yet

2 participants