Skip to content

Disable alteration UI actions for Index Mgmt page#9074

Open
PhDuck wants to merge 2 commits into
mainfrom
private/magram/IndexMgtPageRemoveAlterationUIElements
Open

Disable alteration UI actions for Index Mgmt page#9074
PhDuck wants to merge 2 commits into
mainfrom
private/magram/IndexMgtPageRemoveAlterationUIElements

Conversation

@PhDuck

@PhDuck PhDuck commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Kennie pointed out the UI alteration actions make no sense, they also throw errors or lead to strange behavours:
image

So disable them (grey them out):
image

Fixes AB#641404

Mads Gram and others added 2 commits July 3, 2026 12:53
…page

Set InsertAllowed, DeleteAllowed, and Editable to false on page 8705 "Table Information Card" so the New, Delete, and Edit actions no longer appear.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dropdown

Copilot incorrectly suggested to use Editable = "false" which breaks the
dropdown...
@PhDuck PhDuck requested a review from a team July 3, 2026 13:47
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: no-knowledge

The PR changes a single line-property addition (DeleteAllowed/InsertAllowed/ModifyAllowed = false) on src/System Application/App/Table Information/src/TableInformationCard.Page.al. No knowledge article in performance, security, privacy, upgrade, style, or ui domains addresses this specific page-property pattern after keyword narrowing against the diff, so every sub-skill's worklist is empty.

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

No findings were posted for this iteration.

Orchestrator pre-filter (13 file(s) excluded)

  • layer-disabled (knowledge) : 13 file(s)

Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.

@PhDuck PhDuck requested a review from BardurKnudsen July 3, 2026 14:01

@BardurKnudsen BardurKnudsen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pulcher.

@PhDuck PhDuck enabled auto-merge July 3, 2026 14:22
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