Skip to content

Scheduler: Add Appointment Form Customization to scheduler.d.ts#31127

Merged
sjbur merged 22 commits intoDevExpress:25_2from
sjbur:issue-1784_25_2
Oct 6, 2025
Merged

Scheduler: Add Appointment Form Customization to scheduler.d.ts#31127
sjbur merged 22 commits intoDevExpress:25_2from
sjbur:issue-1784_25_2

Conversation

@sjbur
Copy link
Contributor

@sjbur sjbur commented Sep 22, 2025

No description provided.

@sjbur sjbur self-assigned this Sep 22, 2025
@sjbur sjbur added the 25_2 label Sep 22, 2025
@sjbur sjbur marked this pull request as ready for review September 22, 2025 10:57
@sjbur sjbur requested review from a team as code owners September 22, 2025 10:57
// Form
// Toolbar
// Options
type IItemProps = React.PropsWithChildren<{
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it expected that toolbar item props have been merged with form item props?
Looks like it can cause inadequate props typing

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, this is expected behavior. In metadata we merge complex options (configuration components) with same option name (item). I agree, it looks bad here. For better typing specific form items are available (SimpleItem, GroupItem, TabbedItem, etc)

sjbur and others added 10 commits September 24, 2025 10:41
…_25_2

# Conflicts:
#	packages/devextreme-angular/src/ui/scheduler/index.ts
#	packages/devextreme-angular/src/ui/scheduler/nested/editing.ts
#	packages/devextreme-react/src/scheduler.ts
#	packages/devextreme/js/ui/scheduler.d.ts
#	packages/devextreme/ts/dx.all.d.ts
# Conflicts:
#	packages/devextreme-angular/src/ui/scheduler/index.ts
@mpreyskurantov mpreyskurantov changed the title Scheduler: Add public API in scheduler.d.ts Scheduler: Add public API for form in scheduler.d.ts Oct 2, 2025
@sjbur sjbur changed the title Scheduler: Add public API for form in scheduler.d.ts Scheduler: Add Appointment Form Customization to scheduler.d.ts Oct 2, 2025
@sjbur sjbur merged commit 61d6b3f into DevExpress:25_2 Oct 6, 2025
395 of 396 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants