Skip to content

Conversation

@mddifilippo89
Copy link
Member

No description provided.

mdd-update-bing-sample
mdd-update-display-all
@mddifilippo89 mddifilippo89 added the status: in-review PR ready for review label Nov 19, 2025
Copy link
Collaborator

@HUSSAR-mtrela HUSSAR-mtrela left a comment

Choose a reason for hiding this comment

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

some changes are needed

this.geoMap.backgroundContent = this.getBingMapsImagery(BingMapsImageryStyle.Road);

// Azure
if (text.startsWith("AzureMaps")) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be "ELSE IF" instead of IF condition

if (text.startsWith("AzureMaps")) {

if (!this.azureKey) {
const url = this.placeholderMap[text];
Copy link
Collaborator

Choose a reason for hiding this comment

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

add this code:

        if (location.protocol !== "https:") {
            uri = uri.replace("https:", "http:");
        }

@HUSSAR-mtrela HUSSAR-mtrela merged commit 2b8ad84 into vnext Nov 20, 2025
5 checks passed
@HUSSAR-mtrela HUSSAR-mtrela deleted the mdd-update-bing-samples branch November 20, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: in-review PR ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants