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

When controlling field visibility, use variable instead of field to avoid client error #635

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

JesperSchulz
Copy link
Contributor

@JesperSchulz JesperSchulz commented Feb 26, 2024

Summary

  1. Go to Retention Policies
  2. Create new one or edit

Expected: works
Actual:
Copy the error details to share with support.:

The identifier 'Apply to all records' could not be found.

Internal session ID:
1fcd0b6e-05f9-457a-8bf2-31f72d10bb64

Application Insights session ID:
fac2fc19-9b4b-447b-86a7-770b14696b3a

Client activity id:
7079d263-1e87-fcb5-e9f2-d38e66b0eaff

Time stamp on error:
2024-01-24T13:37:55.5870243Z

User telemetry id:
00000000-0000-0000-0000-000000000000

Client workload elaborates: The issue is the addition of the two fields and the visible property referencing Rec."Apply to all records".

Work Item(s)

Fixes AB#497821

Copy link

Issue #497821 is not valid. Please make sure you link an issue that exists, is open and is approved.

@github-actions github-actions bot added this to the Version 25.0 milestone Feb 26, 2024
@JesperSchulz JesperSchulz enabled auto-merge (squash) February 26, 2024 12:50
@JesperSchulz JesperSchulz merged commit c3dfea3 into main Feb 26, 2024
28 of 29 checks passed
@JesperSchulz JesperSchulz deleted the bugs/497821-CannotEditRetentionPolicies branch February 26, 2024 12:55
JesperSchulz added a commit that referenced this pull request Feb 28, 2024
…ld to avoid client error (#655)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Backport of  #635

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#502379](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/502379)
JesperSchulz added a commit that referenced this pull request Feb 28, 2024
…ld to avoid client error (#656)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Backport of #635

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#502380](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/502380)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants