New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #174, allowing specific days of the week to be disabled. #2147
Conversation
…ific yyyy-mm-dd format strings
I forced push to my master and I'm not 100% that's what made TravicCI mad, but it was coincidental. Happy to adjust, just let me know the best path to resolution. |
R/input-date.R
Outdated
#' ) | ||
#' | ||
#' shinyApp(ui, server = function(input, output) { }) | ||
#' } | ||
#' @export | ||
dateInput <- function(inputId, label, value = NULL, min = NULL, max = NULL, | ||
format = "yyyy-mm-dd", startview = "month", weekstart = 0, language = "en", | ||
format = "yyyy-mm-dd", startview = "month", weekstart = 0, | ||
datesdisabled = NULL, daysofweekdisabled = NULL, language = "en", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you move these arguments to the end? Even though this is the right place to put them conceptually, there could be existing code out there that is broken by this change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing!
Can you send a CLA, as described in https://github.com/rstudio/shiny/blob/master/CONTRIBUTING.md? |
…l if datesdisabled is Date object
CLA sent. Updated argument ordering and added the same conditional formatting for |
Fixes #174, with a new parameter
daysofweekdisabled
available todataInput()
.An example app:
Original code by Niko Eckerskorn (@kadrach). I made this PR because the original fork was 2 years behind
master
and the issue is still unresolved.