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

Update ConnectivityAppDefinitions.Codeunit.al #25403

Merged
merged 4 commits into from
Feb 5, 2024

Conversation

lukasaus
Copy link
Contributor

@lukasaus lukasaus commented Dec 8, 2023

Three new qualifying countries added: ES, FI, LV.

As per statistics:
image

Fixes AB#498782

Three new qualifying countries added: ES, FI, LV.
@lukasaus lukasaus requested a review from a team as a code owner December 8, 2023 14:23
@lukasaus lukasaus requested a review from henrikfrovst December 8, 2023 14:23
@JesperSchulz JesperSchulz added the connectivity-apps PRs for adding Connectivity Apps label Dec 13, 2023
@sorenfriisalexandersen
Copy link

@lukasaus I am sorry, but we cannot yet confirm the 3 customers in ES and FI. I suspect this has to do with our own refresh of the data (currently looking at data that is 22 days old). Let me see if we can get it refreshed on our side so we see the same. Stay tuned.

@JesperSchulz @haoranpb This PR is fine in theory - just need to be able to confirm the numbers before we approve.

@sorenfriisalexandersen
Copy link

@lukasaus I can confirm ES and LV now, but I still don't see any installs in FI, even when looking at fresh telemetry from yesterday.
Could you send me the tenant ID's in an email, so I can look into it? Thanks.

@JesperSchulz FYI

@JesperSchulz JesperSchulz added processing-PR The PR is currently being reviewed linked Issue is linked to a Azure Boards work item labels Feb 5, 2024
@JesperSchulz JesperSchulz enabled auto-merge (squash) February 5, 2024 14:28
@JesperSchulz JesperSchulz merged commit 9d5892b into microsoft:main Feb 5, 2024
6 checks passed
@sorenfriisalexandersen
Copy link

sorenfriisalexandersen commented Feb 5, 2024 via email

@lukasaus lukasaus deleted the patch-4 branch February 5, 2024 15:25
@JesperSchulz JesperSchulz added the ships-in-future-update Fix ships in a future update label Feb 6, 2024
@JesperSchulz
Copy link
Contributor

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Build ID: 15971.

aholstrup1 pushed a commit to aholstrup1/ALAppExtensions that referenced this pull request Sep 6, 2024
Three new qualifying countries added: ES, FI, LV.

As per statistics:

![image](https://github.com/microsoft/ALAppExtensions/assets/130442161/787798cb-7397-4639-aec2-3a13df9958c8)

Fixes AB#498782

---------

Co-authored-by: Jesper Schulz-Wedde <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connectivity-apps PRs for adding Connectivity Apps linked Issue is linked to a Azure Boards work item processing-PR The PR is currently being reviewed ships-in-future-update Fix ships in a future update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants