Skip to content

Commit

Permalink
Fixes #3597 first day of week on calendar transaction (#3598)
Browse files Browse the repository at this point in the history
* fix 3597 first day of week on calendar transactin

* added release note

* fixed release note filename
  • Loading branch information
attyluccio authored Oct 7, 2024
1 parent a7b8d12 commit e653ad3
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
3 changes: 1 addition & 2 deletions packages/desktop-client/src/components/select/DateSelect.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ import {
getShortYearRegex,
currentDate,
} from 'loot-core/src/shared/months';
import { stringToInteger } from 'loot-core/src/shared/util';

import { useSyncedPref } from '../../hooks/useSyncedPref';
import { styles, theme } from '../../style';
Expand Down Expand Up @@ -134,7 +133,7 @@ const DatePicker = forwardRef<DatePickerForwardedRef, DatePickerProps>(
picker.current = new Pikaday({
theme: 'actual-date-picker',
keyboardInput: false,
firstDay: stringToInteger(firstDayOfWeekIdx),
firstDay: parseInt(firstDayOfWeekIdx),
defaultDate: value
? parse(value, dateFormat, currentDate())
: currentDate(),
Expand Down
6 changes: 6 additions & 0 deletions upcoming-release-notes/3598.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
category: Bugfix
authors: [attyluccio]
---

Fixes First day of the week option not being used correctly in the calendar when entering a new transaction

0 comments on commit e653ad3

Please sign in to comment.