Conversation
There was a problem hiding this comment.
Do you really need this file?
There was a problem hiding this comment.
Same problem with URL ending in '&'
… used enums for types
MarielaTihova
left a comment
MarielaTihova
left a comment
There was a problem hiding this comment.
Please resolve comments
There was a problem hiding this comment.
This file is not needed
There was a problem hiding this comment.
Shouldn't we make a new request when we scroll to see the remaining data in the grid instead of fetching everything at once?
There was a problem hiding this comment.
Shouldn't we keep the behaviour consistent here? For example look at this sample for Angular here https://www.infragistics.com/angular-demos/grid/grid-sample-4. A new request is being made on every scroll.
There was a problem hiding this comment.
I suppose it is the ngAfterViewInit in angular that makes that possible, but you will get empty grid if we remove the initial load. Do you have a suggestion?
There was a problem hiding this comment.
📌 Putting a pin on this to discuss more about the behaviour.
There was a problem hiding this comment.
This produces an error. Sample doesn't run. Please check it
…nd request cancellation
There was a problem hiding this comment.
NOT the "Filtering Options feature"
There was a problem hiding this comment.
NOT the "Filtering Options feature"
There was a problem hiding this comment.
NOT the "Filtering Options feature"
There was a problem hiding this comment.
Shouldn't we keep the behaviour consistent here? For example look at this sample for Angular here https://www.infragistics.com/angular-demos/grid/grid-sample-4. A new request is being made on every scroll.
There was a problem hiding this comment.
📌 Putting a pin on this to discuss more about the behaviour.
There was a problem hiding this comment.
I don't think this archived file should be here.
|
@tishko0 Please resolve remaining comments |
Virtualization sample is okay now. The unique values is still work in progress so I will update that one later when I can see what can be done in angular for it to work properly. For now it's just a placeholder. |
There was a problem hiding this comment.
By passing these parameters the request is always getting top=50 so the records from 50 to 77 (total nu,ber of records) are not shown. We are able to scroll in the grid up to 77th record but it shows empty rows
|
@mddragnev can you have a look at this as well? I remember discussing this behaviour with you and others from the team, but I don't remember the exact advice you gave @tishko0 so I might have missed something in my last commit. |
3 participants
No description provided.