981174: Add Grid with AI Assist view sample#27
Open
Vignesh050801 wants to merge 12 commits intomasterfrom
Open
981174: Add Grid with AI Assist view sample#27Vignesh050801 wants to merge 12 commits intomasterfrom
Vignesh050801 wants to merge 12 commits intomasterfrom
Conversation
rajapandiyan-settu
requested changes
Oct 6, 2025
rajapandiyan-settu
requested changes
Oct 6, 2025
rajapandiyan-settu
requested changes
Oct 6, 2025
| const toolbarOptions: object[] = [{ tooltipText: 'AI Assist', prefixIcon: 'e-assistview-icon', id: 'ai-assist-btn', align: 'Left' }]; | ||
|
|
||
| // Handles the Grid toolbar button click action. If the AI Assist button clicked shows the AI Assist dialog. | ||
| const toolbarClick = (args: any) => { |
Collaborator
There was a problem hiding this comment.
don't use any type here
Collaborator
Author
There was a problem hiding this comment.
we need to use any bcoz of eventTarget possibly null.
|
|
||
| // Handles prompt request execution. | ||
| const onPromptRequest = (args: PromptRequestEventArgs) => { | ||
| (assistView as any).stopResponding.classList.remove('e-btn-active'); |
Collaborator
There was a problem hiding this comment.
don't use any here
Collaborator
Author
There was a problem hiding this comment.
We need to use any type here bcoz of accessing the private method
rajapandiyan-settu
requested changes
Oct 6, 2025
Collaborator
rajapandiyan-settu
left a comment
There was a problem hiding this comment.
provide a proper PR template for this implementation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug description
Need to add more sample with ai grid and planned for sp release.
Root cause
Added new sample
Reason for not identifying earlier
Find how it was missed in our earlier testing and development by analyzing the below checklist. This will help prevent similar mistakes in the future.
Guidelines/documents are not followed
Common guidelines / Core team guideline
Specification document
Requirement document
Guidelines/documents are not given
Common guidelines / Core team guideline
Specification document
Requirement document
Reason:
NA
Action taken:
SB automation has been covered in this case.
Related areas:
Sorting, Alignment, Touch, RTL, resizing
Is it a breaking issue?
No.
Solution description
Get response from open ai to generate query for data operation and assigned to grid query.
Output screenshots
NA.
Areas affected and ensured
NA.
Additional checklist
This may vary for different teams or products. Check with your scrum masters.
Did you run the automation against your fix? - No.
Is there any API name change? - No
Is there any existing behavior change of other features due to this code change? - No
Does your new code introduce new warnings or binding errors? - No
Does your code pass all FxCop and StyleCop rules? a- No
Did you record this case in the unit test or UI test? - No.