Skip to content

Update calendar.md #446

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

Closed
wants to merge 2 commits into from
Closed

Update calendar.md #446

wants to merge 2 commits into from

Conversation

rogercreagh
Copy link
Contributor

@rogercreagh rogercreagh commented Apr 25, 2025

User description

Added note about setting default date to now


PR Type

documentation


Description

  • Added documentation for the default attribute in calendar field.

  • Explained how to set the default date to the current date.


Changes walkthrough 📝

Relevant files
Documentation
calendar.md
Add documentation for calendar field default date               

versioned_docs/version-5.2/general-concepts/forms-fields/standard-fields/calendar.md

  • Added explanation for the default attribute in calendar fields.
  • Noted that default="now" sets the date to the current date.
  • +2/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • Added note about setting default date to now
    Copy link
    Contributor

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    Copy link
    Contributor

    qodo-merge-pro bot commented Apr 25, 2025

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @rogercreagh
    Copy link
    Contributor Author

    Sorry about separate commits for the separate versions 5.2 and 5.3 - should have been one with both changes. Hey Ho.

    Further clarified the default attribute description in response to a comment
    @HLeithner
    Copy link
    Member

    I'm closing this because 5.2 is not longer supported and you have a pr for current next branch

    @HLeithner HLeithner closed this Apr 26, 2025
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants