Skip to content

ECWID-109995 Custom Order Statuses: new API for custom order statuses - #236

Merged
elviswarlock merged 6 commits into
masterfrom
ECWID-109995
Dec 13, 2022
Merged

ECWID-109995 Custom Order Statuses: new API for custom order statuses#236
elviswarlock merged 6 commits into
masterfrom
ECWID-109995

Conversation

@winrokru

@winrokru winrokru commented Nov 27, 2022

Copy link
Copy Markdown
Contributor

Api для ECWID-109995 Custom Order Statuses: API /order_statuses

@winrokru
winrokru force-pushed the ECWID-109995 branch 2 times, most recently from d350768 to f98adf7 Compare November 27, 2022 12:14
@Alexis2004
Alexis2004 force-pushed the master branch 2 times, most recently from f9d569e to efa20a5 Compare November 29, 2022 10:02
elviswarlock
elviswarlock requested changes Dec 12, 2022
View reviewed changes

@elviswarlock elviswarlock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Лучше заменить использование Setting -> Settings, так как у нас много настроек и в единственном числе лучше не употреблять.
А точно у нас в API поля: defaultStatus, name, nameTranslations должны быть nullable?

@winrokru

winrokru commented Dec 12, 2022

Copy link
Copy Markdown
Contributor Author

Лучше заменить использование Setting -> Settings, так как у нас много настроек и в единственном числе лучше не употреблять. А точно у нас в API поля: defaultStatus, name, nameTranslations должны быть nullable?

1)Я в самом Api использую для одного кастомного статуса OrderStatusSetting , если они приходят в списке то
OrderStatusesSetting , то есть у мне множественное число Statuses, если дословно настройка статусов заказов.
Все сделано в единой стилистике и в api и vue, поэтому лучше оставить так.
2)Да эти поля: defaultStatus, name, nameTranslations могут быть nullable , так как можем передавать не все , а одно из них

@winrokru

winrokru commented Dec 12, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

то есть за основу взял entity OrderStatusSetting.kt в базе таблица "order_status_setting" (настройка статуса заказа), если возвращаем список этих самых entity , то юзаем getOrderStatusesSetting (получить настройку статусов заказа)

@elviswarlock
elviswarlock merged commit f377124 into master Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elviswarlock elviswarlock elviswarlock approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@winrokru @elviswarlock