Skip to content

refactor(Scheduler): rename _getAppointmentTemplate and _checkRecurringAppointment#32826

Closed
aleksei-semikozov wants to merge 6 commits intoDevExpress:26_1from
aleksei-semikozov:3518-rename-getAppointmentTemplate-checkRecurring
Closed

refactor(Scheduler): rename _getAppointmentTemplate and _checkRecurringAppointment#32826
aleksei-semikozov wants to merge 6 commits intoDevExpress:26_1from
aleksei-semikozov:3518-rename-getAppointmentTemplate-checkRecurring

Conversation

@aleksei-semikozov
Copy link
Contributor

Addresses review comments from sjbur on PR #32811.

Changes

  • _getAppointmentTemplategetAppointmentTemplate in m_scheduler.ts
  • _checkRecurringAppointmentcheckRecurringAppointment in m_scheduler.ts
  • Updated callers in m_compact_appointments_helper.ts, m_subscribes.ts, and integration.appointmentCollector.tests.js

Related

Part of #32811 (3518-scheduler-core-replace-underscore-v2)

…scheduler.ts

- _appointmentPopup → appointmentPopup
- _appointmentTooltip → appointmentTooltip
- _workSpaceRecalculation → workSpaceRecalculation
- _header → header
- _reloadDataSource → reloadDataSource
- _refreshWorkSpace → refreshWorkSpace
- _setRemoteFilterIfNeeded → setRemoteFilterIfNeeded
- _renderWorkSpace → renderWorkSpace
…enames

- helpers.js: _appointmentPopup → appointmentPopup
- integration.dateNavigator.tests.js: update assert message for setRemoteFilterIfNeeded
- common.options.tests.js: update assert message for reloadDataSource
…ests

- common.methods.tests.js: _editAppointmentData → editAppointmentData
- common.options.tests.js: _createAppointmentPopupForm → createAppointmentPopupForm
…ngAppointment

- m_scheduler.ts: _getAppointmentTemplate → getAppointmentTemplate
- m_scheduler.ts: _checkRecurringAppointment → checkRecurringAppointment
- m_compact_appointments_helper.ts: update instance call
- integration.appointmentCollector.tests.js: update mock method name
- m_subscribes.ts: update 2 calls to checkRecurringAppointment
@aleksei-semikozov aleksei-semikozov self-assigned this Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants