Skip to content

Conversation

@sfc-gh-bnisco
Copy link
Collaborator

@sfc-gh-bnisco sfc-gh-bnisco commented Oct 22, 2024

Describe your changes

  • st.date_input now understands the user's current locale and sets the start of the week based on it.

GitHub Issue Link (if applicable)

Closes #5215

Testing Plan

  • Explanation of why no additional tests are needed
  • Unit Tests (JS and/or Python) ✅
  • E2E Tests ✅
  • Any manual testing needed?

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@sfc-gh-bnisco sfc-gh-bnisco added feature:st.date_input Related to the `st.date_input` widget do-not-merge PR is blocked from merging change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users security-assessment-completed Security assessment has been completed for PR labels Oct 22, 2024
@sfc-gh-bnisco sfc-gh-bnisco force-pushed the bnisco/localize-calendar branch from 7abd6fb to 3d8baa8 Compare October 28, 2024 21:35
@sfc-gh-bnisco sfc-gh-bnisco changed the title [WIP] [feat] Localize Calendar in st.date_input [feat] Localize calendar week start date in st.date_input Oct 29, 2024
@sfc-gh-bnisco sfc-gh-bnisco removed the do-not-merge PR is blocked from merging label Oct 29, 2024
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: This was previously an existing transitive dependency. I added it here to make it explicit / ensure the eslint rule about dependencies is happy.

@sfc-gh-bnisco sfc-gh-bnisco marked this pull request as ready for review October 29, 2024 17:06
@sfc-gh-bnisco sfc-gh-bnisco force-pushed the bnisco/localize-calendar branch from d8ff62d to 27f4805 Compare October 30, 2024 19:55
Copy link
Collaborator

@lukasmasuch lukasmasuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@sfc-gh-bnisco sfc-gh-bnisco merged commit 5bf6594 into develop Oct 30, 2024
@sfc-gh-bnisco sfc-gh-bnisco deleted the bnisco/localize-calendar branch October 30, 2024 22:42
edegp pushed a commit to edegp/streamlit that referenced this pull request Jan 19, 2025
…9706)

## Describe your changes

- `st.date_input` now understands the user's current locale and sets the
start of the week based on it.

## GitHub Issue Link (if applicable)

Closes streamlit#5215 

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python) ✅
- E2E Tests ✅
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:feature PR contains new feature or enhancement implementation feature:st.date_input Related to the `st.date_input` widget impact:users PR changes affect end users security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

st.date_input: Starting day in a week

3 participants