You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
В сервисе syncer, нужно добавить возможность, пакетной отправки (batchUpdate) записей для синхронизации. Эта возможность должна быть опциональной, для этого можно использовать параметр options в методе syncUp.
В методе _changesForRecord нужно убрать лишний запрос, который используется для заполнения связей. От записи нужен только первичный ключ, поэтому, вместо запроса, можно создавать запись в сторе (#130).
Метод syncUp может принимать в первом параметре массив моделей аудита, по которым должна быть выполнена синхронизация, но сейчас это не рабочий сценарий. Нужно это исправить, думаю, также можно добавить в сервис публичный метод, для получения всех записей аудита.
The text was updated successfully, but these errors were encountered:
Ещё есть проблема с реализацией синхронизации с использованием batchUpdate и использованием настроек сервиса по умолчанию.
По умолчанию, для каждого изменения записи, создается отдельная запись на синхронизацию, и при формировании запросов для синхронизации возникают конфликты, когда в запрос пытаются попасть несколько состояний для одной записи (создание и изменение, несколько изменений, и т.д.).
В сервисе
syncer
, нужно добавить возможность, пакетной отправки (batchUpdate
) записей для синхронизации. Эта возможность должна быть опциональной, для этого можно использовать параметрoptions
в методеsyncUp
.В методе
_changesForRecord
нужно убрать лишний запрос, который используется для заполнения связей. От записи нужен только первичный ключ, поэтому, вместо запроса, можно создавать запись в сторе (#130).Метод
syncUp
может принимать в первом параметре массив моделей аудита, по которым должна быть выполнена синхронизация, но сейчас это не рабочий сценарий. Нужно это исправить, думаю, также можно добавить в сервис публичный метод, для получения всех записей аудита.The text was updated successfully, but these errors were encountered: