[REACT] finalizeCellEdit

Our blazing fast Grid component built with pure JavaScript


Post Reply
User avatar
germandsantana
Premium Member
Premium Member
Posts: 4
Joined: Thu Jun 18, 2020 7:44 pm

[REACT] finalizeCellEdit

Post by germandsantana »

Hi,

We are using the finalizeCellEdit function to perfom async validations .
When the async function takes much time to run and the user clicks on another cell during the execution , an error is thrown:

Unhandled Rejection (TypeError): editorContext is null
onEditorComplete
grid-3.1.5/build/grid.umd.js:73382
trigger
grid-3.1.5/build/grid.umd.js:40139

This is the code sample to reproduce it:

Code: Select all

  finalizeCellEdit:async (context) => {
                      await new Promise(resolve => setTimeout(resolve, 3000));
                        	console.log("cellEdit");
                        	return true;
                      }

Thanks,

Germán Santana


User avatar
mats
Core Developer
Core Developer
Posts: 16347
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: [REACT] finalizeCellEdit

Post by mats »

Reproduced, we'll investigate: https://github.com/bryntum/support/issues/963

Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

User avatar
germandsantana
Premium Member
Premium Member
Posts: 4
Joined: Thu Jun 18, 2020 7:44 pm

Re: [REACT] finalizeCellEdit

Post by germandsantana »

Hi,

Do you know if this issue could be fixed in a soon release of the grid?

We have an active license of the grid. I am posting with my email personal account, because registration on the forum with my work email account was nearly impossible , seems like office 365 is blocking all messages from the forum.

My email is german.santana@hydra-management.com

Thanks


User avatar
Maxim Gorkovsky
Core Developer
Core Developer
Posts: 3184
Joined: Wed Jan 08, 2014 11:46 am

Re: [REACT] finalizeCellEdit

Post by Maxim Gorkovsky »

It is scheduled for next grid patch release.


Post Reply