Skip to content

Control fields visibility for Agent Model list in lookup mode (#8209)#8261

Open
mbaranovych wants to merge 1 commit into
releases/28.xfrom
private/mbaranovych/cherryLookup
Open

Control fields visibility for Agent Model list in lookup mode (#8209)#8261
mbaranovych wants to merge 1 commit into
releases/28.xfrom
private/mbaranovych/cherryLookup

Conversation

@mbaranovych
Copy link
Copy Markdown
Contributor

@mbaranovych mbaranovych commented May 21, 2026

What & why

We want to show only model id, name and availability when opening page with lookup from agent setup page. After changes lookup looks like below while main page still shows all fields.

image

Linked work

Fixes #

AB#635864

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

What & why

Linked work

Fixes #

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

<!--
Thanks for contributing to BCApps!

A few things before you hit "Create pull request":
- Your PR must link to an approved issue. New here? See CONTRIBUTING.md.
- You must have built and run your change yourself. CI is a safety net,
not a substitute.
- If you used AI or an agent to write this PR, you are still the author.
Read the diff,
  build it, and try it before requesting review.

Contributing guide:
https://github.com/microsoft/BCApps/blob/main/CONTRIBUTING.md
Local dev environment:
https://github.com/microsoft/BCApps/blob/main/LOCAL_DEV_ENV.md
-->

## What & why

<!-- A few sentences: what does this change do, and what problem does it
solve? -->
We want to show only model id, name and availability when opening page
with lookup from agent setup page. After changes lookup looks like below
while main page still shows all fields.

<img width="1300" height="441" alt="image"
src="https://github.com/user-attachments/assets/e1eeaad7-1bf5-4538-81e3-b4cf814170f1"
/>


## Linked work

<!-- Required: link an approved GitHub issue using "Fixes #<number>".
Microsoft contributors: also link the ADO work item with "AB#<number>"
if you have one. -->

Fixes #


[AB#635864](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/635864)

## How I validated this

- [X] I read the full diff and it contains only changes I intended.
- [X] I built the affected app(s) locally with no new analyzer warnings.
- [X] I ran the change in Business Central and confirmed it behaves as
expected.
- [ ] I added or updated tests for the new behavior, or explained below
why none are needed.

**What I tested and the outcome** *(required — be specific: scenarios,
commands, screenshots for UI changes)*

<!-- Example:
- Ran the new "Post and Send" action on a sales invoice in a fresh
container; document posted and email queued (see screenshot).
- New unit tests in MyFeatureTest.Codeunit.al pass locally; full module
test suite green.
- No tests added because change is comment-only / refactor with existing
coverage. -->

## Risk & compatibility

<!-- Anything reviewers should watch for: breaking changes, upgrade/data
impact, permissions,
telemetry, feature flags, follow-up work. Write "None" if there's
nothing to call out. -->
@mbaranovych mbaranovych requested a review from a team as a code owner May 21, 2026 08:28
@github-actions github-actions Bot added this to the Version 28.2 milestone May 21, 2026
@mbaranovych mbaranovych enabled auto-merge (squash) May 21, 2026 08:32
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.

2 participants