Skip to content

Conversation

@cpsievert
Copy link
Contributor

Introduced by #149. I'm pretty sure the DBI::SQL() isn't needed in the first place, so we can just drop it.

Here's a reprex:

library(shiny)
library(bslib)
library(querychat)
library(palmerpenguins)

qc <- QueryChat$new(penguins)

ui <- page_sidebar(
  sidebar = qc$sidebar(),
  dataTableOutput("table")
)

server <- function(input, output, session) {
  qc_vals <- qc$server()
  
  output$table <- renderDataTable({
    qc_vals$df()
  })
}

shinyApp(ui, server)
Warning: Error in initialize: cannot use object of class “NULL” in new():  class “SQL” does not extend that class

@cpsievert
Copy link
Contributor Author

@gadenbuie merging to un-break main, but please comment if you'd like

@cpsievert cpsievert merged commit baf0768 into main Dec 9, 2025
10 checks passed
@cpsievert cpsievert deleted the fix/r-null-query branch December 9, 2025 23:27
@github-actions github-actions bot restored the fix/r-null-query branch December 9, 2025 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants