This repository has been archived by the owner on May 15, 2024. It is now read-only.
AndroidX #1095
Merged
AndroidX #1095
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Would be worth writing up some info in the readme.txt about AndroidX I think....... |
mattleibow
previously approved these changes
Feb 10, 2020
| @@ -185,7 +185,11 @@ public void Get_Existing_External_Cache_Shareable_Uri(FileProviderLocation locat | |||
| public void Get_Existing_External_Shareable_Uri(FileProviderLocation location) | |||
| { | |||
| // Save an external directory file | |||
|
|
|||
| #if !__ANDROID_29__ | |||
| var externalRoot = AndroidEnvironment.ExternalStorageDirectory.AbsolutePath; | |||
There was a problem hiding this comment.
Is this deprecated in API 29? Is there an alternative we should be using?
| @@ -13,9 +18,9 @@ namespace Xamarin.Essentials | |||
| Exported = false, | |||
| GrantUriPermissions = true)] | |||
| [MetaData( | |||
| "android.support.FILE_PROVIDER_PATHS", | |||
| "android.support.FILE_PROVIDER_PATHS", // IMPORTANT: This string doesn't change with AndroidX | |||
This is just the compile targets, it still runs all the way to 4.4 - and probably earlier
mattleibow
reviewed
Feb 14, 2020
| @@ -1,6 +1,6 @@ | |||
| <Project Sdk="MSBuild.Sdk.Extras/2.0.54"> | |||
| <PropertyGroup> | |||
| <TargetFrameworks>netstandard1.0;netstandard2.0;Xamarin.iOS10;Xamarin.TVOS10;Xamarin.WatchOS10;MonoAndroid60;MonoAndroid70;MonoAndroid71;MonoAndroid80;MonoAndroid81;MonoAndroid90;MonoAndroid10.0;tizen40;</TargetFrameworks> | |||
| <TargetFrameworks>netstandard1.0;netstandard2.0;Xamarin.iOS10;Xamarin.TVOS10;Xamarin.WatchOS10;MonoAndroid80;MonoAndroid81;MonoAndroid90;MonoAndroid10.0;tizen40;</TargetFrameworks> | |||
There was a problem hiding this comment.
Dropping the 6.0 to 7.1 as this references really old Android Support. It will still RUN on older devices, but we really shouldn't be compiling for this. The tools are really old.
mattleibow
approved these changes
Feb 19, 2020
Redth
added a commit
that referenced
this pull request
Feb 27, 2020
* WebAuthenticator API (#1062) * Initial work for WebAuthenticator API * Fix page loaded in sample * Fix weird generated item * Fix error message * Fix filename, too many .'s * Remove unnecessary comment * Tweak AppleSignIn API * Cancel any previous waiting tasks * Use WebAuthenticationBroker on UWP * Make code more readable * iOS10+ is only supported so remove check * Move last fallback into TVOS * Remove unnecessary duplicate attribute * Change exception type. * Better error message * Make AppleSignIn accessible from shared code * Added sample aspnet project for auth * Apple Signin for iOS only for now * Make auth sample use sample server * Added some tests * Make some methods private that shouldn't be public * Add docs * Increase device test timeout * Add human interaction traits to some new tests * Update devicetests cake Updated to newer addins Stop using cake bootstrappers * Run device tests with cake dotnet core * Bump xunit device runner * Tests go back to netcore 2 * Bump xunit device runner in rest of devicetest projs * Build that android app first! * Cleanup web auth - Change result type name - Clean up result type properties - Remove IsSupported from apple public api - Throw not supported in public apple api on < iOS 13 * Update sample to check for iOS 13 independently * Update docs * Make code a bit more readable. * More readable code changes * Fix test * Add some detection of callback activity This adds some code to ensure the developer has subclassed `WebAuthenticatorCallbackActivity` and registered an appropriate intentfilter for it based on the `callbackUrl` being used. Co-authored-by: James Montemagno <james.montemagno@gmail.com> * Use iOS 13.3 :) (#1089) * Rename apple signin options (#1094) * Rename type * Fix docs * Android setup Script (#1086) * Add Android setup script Installs everything you need for development :) * Update README.md * Update README.md * This double is not in the interning list of double, so this will allocate less. Verified in Benchmark.Net (#1097) * Try catch getting Network Adapter and only query it once. Log if there is an issue. (#1088) Co-authored-by: Jonathan Dick <jondick@gmail.com> * Cache device type and name (#1087) Co-authored-by: Jonathan Dick <jondick@gmail.com> * AndroidX (#1095) * Use AndroidX references for Android 10+ * Directly use AndroidX api's on API 29 + * Bump sample/device tests to api 29 * Migrate resource files in device tests / sample to androidx * Fix nuget references * Fix test with api 29 deprecations * Go STABLE!!! * Drop Android 6.0-7.1 as targets This is just the compile targets, it still runs all the way to 4.4 - and probably earlier Co-authored-by: Matthew Leibowitz <mattleibow@live.com> * four more minor changes (#1101) * This double is not in the interning list of double, so this will allocate less. Verified in Benchmark.Net * Should both be readonly * I reused the branch so I need to remove this for merging * type should be static, features should be unique. more efficient Co-authored-by: Jonathan Dick <jondick@gmail.com> * Launch adjacent flag on Android 24+ (#1074) * Launch adjacent! <3 Dou It! * refactor code * not share * 1.5.0 Changes (#1124) * Implement vertical accuracy in GeoLocation API (#1103) * Location: add property 'VerticalAccuracy' (#1099) * vertical accuracy is only available in Android API level 26 and above (#1099) * update Samples app to show vertical accuracy on GeolocationPage * add missing documentation bits for Location.VerticalAccuracy * .gitattributes: get better diff context for C# code (#1115) * Location.VerticalAccuracy: add runtime check for Android version (#1099) (#1116) * the compile-time check is not enough * it crashed on old Android versions (<8.0) * GH-1102 Fix launcher on older devices (#1120) * Update Launcher.ios.tvos.cs * OpenUrlAsync was introduced in iOS 10 not 12 https://docs.microsoft.com/en-us/dotnet/api/uikit.uiapplication.openurlasync?view=xamarin-ios-sdk-12 * Fix trailing whitespace * Really? fix whitespace * Really! Fix whitespace Co-authored-by: Janus Weil <janus@gcc.gnu.org> Co-authored-by: James Montemagno <james.montemagno@gmail.com> Co-authored-by: James Montemagno <james.montemagno@gmail.com> Co-authored-by: Niklas Schilli <schillinik@yahoo.de> Co-authored-by: Matthew Leibowitz <mattleibow@live.com> Co-authored-by: Janus Weil <janus@gcc.gnu.org>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Make Essentials compile against AndroidX for MonoAndroid10.0 projects.