Skip to content
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

expiration time selection fix for bootstrap template #1322

Merged
merged 2 commits into from May 13, 2024

Conversation

elrido
Copy link
Contributor

@elrido elrido commented May 9, 2024

This PR fixes expiration time selection for bootstrap template.

  • undue the regression introduced for bootstrap5 (line 4374)
  • document the template specific code paths
  • add an onchange event handler for bootstrap5 & page templates
  • remove unnecessary fallback in line 4572, pasteExpiration is null during the init call, getExpirationDefault will return the currently selected value of the select element - it would only be empty if no expirations got configured, which would cause other breakage

@elrido elrido requested a review from rugk May 12, 2024 08:07
@elrido elrido merged commit b080fd5 into master May 13, 2024
16 of 17 checks passed
@elrido elrido deleted the bootstrap-expiration-time branch May 13, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants