You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've confirmed commit b42dd91 (part of v0.31) causes a breaking change.
In previous versions, when pushing data to a DT via replaceData(), only column order was used (i.e. colnames were ignored).
With b42dd91, colnames are now required (assuming the original 'shell' table being proxied has colnames) and replaceData() calls now fail when any expected colname is missing, triggering a window.alert() on the client.
Perhaps matching on colnames is a perfectly fine API change, but neither the release notes nor the replaceData() docs highlight this change as a new requirement (i.e. a breaking change). I suspect many applications (like some of mine :-)) will fail surprisingly after the DT update.
I have provided the necessary information about my issue.
If I'm asking a question, I have already asked it on Stack Overflow or RStudio Community, waited for at least 24 hours, and included a link to my question there.
If I'm filing a bug report, I have included a minimal, self-contained, and reproducible example, and have also included xfun::session_info('DT'). I have upgraded all my packages to their latest versions (e.g., R, RStudio, and R packages), and also tried the development version: remotes::install_github('rstudio/DT').
If I have posted the same issue elsewhere, I have also mentioned it in this issue.
I have learned the Github Markdown syntax, and formatted my issue correctly.
I understand that my issue may be closed if I don't fulfill my promises.
The text was updated successfully, but these errors were encountered:
I've confirmed commit b42dd91 (part of v0.31) causes a breaking change.
In previous versions, when pushing data to a DT via
replaceData()
, only column order was used (i.e. colnames were ignored).With b42dd91, colnames are now required (assuming the original 'shell' table being proxied has colnames) and
replaceData()
calls now fail when any expected colname is missing, triggering awindow.alert()
on the client.Perhaps matching on colnames is a perfectly fine API change, but neither the release notes nor the
replaceData()
docs highlight this change as a new requirement (i.e. a breaking change). I suspect many applications (like some of mine :-)) will fail surprisingly after the DT update.The below works with 8f6bfd1 but fails with b42dd91.
By filing an issue to this repo, I promise that
xfun::session_info('DT')
. I have upgraded all my packages to their latest versions (e.g., R, RStudio, and R packages), and also tried the development version:remotes::install_github('rstudio/DT')
.I understand that my issue may be closed if I don't fulfill my promises.
The text was updated successfully, but these errors were encountered: