Skip to content

Updating service used for sample on all platforms #1590

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

Merged
merged 4 commits into from
May 14, 2025

Conversation

cgutierrez-gis
Copy link
Collaborator

@cgutierrez-gis cgutierrez-gis commented May 9, 2025

Description

Previously used service has now required a token authentication to access and Sample wasn't working, the service used has been updated based on what the Kotlin sample viewer uses.

Type of change

  • Bug fix

Platforms tested on

  • WPF .NET 8
  • WPF Framework
  • WinUI
  • MAUI WinUI
  • MAUI Android
  • MAUI iOS
  • MAUI MacCatalyst

Checklist

  • Self-review of changes
  • All changes work as expected on all affected platforms
  • There are no warnings related to changes
  • Code is commented and follows .NET conventions and standards
  • Codemaid and XAML styler extensions have been run on every changed file
  • No unrelated changes have been made to any other code or project files
  • Screenshots are correct size and display in description tab (800 x 600 on Windows, MAUI mobile platforms use the MAUI Windows screenshot)

@cgutierrez-gis cgutierrez-gis self-assigned this May 12, 2025
@cgutierrez-gis cgutierrez-gis changed the base branch from v.next to main May 13, 2025 03:15
Copy link
Collaborator

@williambohrmann3 williambohrmann3 left a comment

Choose a reason for hiding this comment

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

Thanks. Test is failing since this branch doesn't have readme fixes I did... just remove the unrelated changes and you should be good.

@@ -1,7 +1,7 @@
<Project>
<PropertyGroup>
<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
<ArcGISMapsSDKVersion Condition="'$(ArcGISMapsSDKVersion)'==''">200.7.0</ArcGISMapsSDKVersion>
<ArcGISMapsSDKVersion Condition="'$(ArcGISMapsSDKVersion)'==''">200.8.0</ArcGISMapsSDKVersion>
Copy link
Collaborator

Choose a reason for hiding this comment

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

unrelated change

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.esri.arcgisruntime.samples.maui" android:versionCode="6" android:versionName="200.7.0">
<manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.esri.arcgisruntime.samples.maui" android:versionCode="7" android:versionName="200.8.0">
Copy link
Collaborator

Choose a reason for hiding this comment

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

unrelated change

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

@cgutierrez-gis
Copy link
Collaborator Author

Thanks. Test is failing since this branch doesn't have readme fixes I did... just remove the unrelated changes and you should be good.

@williambohrmann3 I've regressed the unrelated changes in my branch, good to merge?

@williambohrmann3
Copy link
Collaborator

@cgutierrez-gis yep, good to merge
last failing check is due to nuget status being degraded btw

@cgutierrez-gis cgutierrez-gis merged commit c47c9ba into main May 14, 2025
5 of 6 checks passed
@cgutierrez-gis cgutierrez-gis deleted the Carlos/ShowLabelsLayerServiceUpdate branch May 14, 2025 23:41
github-actions bot pushed a commit that referenced this pull request May 14, 2025
Co-authored-by: William Bohrmann <[email protected]>
Co-authored-by: Carlos Gutierrez <[email protected]>
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