LP-5613 #2

Open
dmitrii.pichenikin wants to merge 37 commits from LP-5613 into dev
No description provided.
dmitrii.pichenikin added 10 commits 2026-03-05 10:22:20 +00:00
dmitrii.pichenikin requested review from kirill.moos 2026-03-05 10:22:20 +00:00
dmitrii.pichenikin requested review from kirill.samusenko 2026-03-05 10:22:21 +00:00
dmitrii.pichenikin added 1 commit 2026-03-10 07:40:32 +00:00
dmitrii.pichenikin added 1 commit 2026-03-10 11:39:51 +00:00
dmitrii.pichenikin added 1 commit 2026-03-10 12:15:27 +00:00
dmitrii.pichenikin added 1 commit 2026-03-10 12:33:52 +00:00
dmitrii.pichenikin added 1 commit 2026-03-10 12:56:06 +00:00
dmitrii.pichenikin added 1 commit 2026-03-11 08:33:55 +00:00
dmitrii.pichenikin added 1 commit 2026-03-11 08:37:42 +00:00
dmitrii.pichenikin added 1 commit 2026-03-11 10:46:01 +00:00
dmitrii.pichenikin added 1 commit 2026-03-11 11:38:55 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 07:49:07 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 11:01:37 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 11:55:56 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 11:57:40 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 12:53:18 +00:00
dmitrii.pichenikin added 1 commit 2026-03-12 13:43:01 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 07:48:13 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 12:28:52 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 13:09:47 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 13:24:27 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 13:28:35 +00:00
dmitrii.pichenikin added 1 commit 2026-03-13 14:37:25 +00:00
dmitrii.pichenikin added 1 commit 2026-03-16 08:31:28 +00:00
dmitrii.pichenikin added 1 commit 2026-03-16 08:44:56 +00:00
dmitrii.pichenikin added 1 commit 2026-03-16 10:56:45 +00:00
dmitrii.pichenikin added 1 commit 2026-03-16 11:09:18 +00:00
dmitrii.pichenikin added 1 commit 2026-03-16 12:01:34 +00:00
dmitrii.pichenikin added 1 commit 2026-03-17 12:00:54 +00:00
vladimir.khablak reviewed 2026-03-18 10:54:25 +00:00
@ -66,0 +170,4 @@
});
}
private doDelete = async (url: string, headers: {[x:string]: string} = {}) => {

По сути код в методах для crud дублиркется. Можно было бы сократить

По сути код в методах для crud дублиркется. Можно было бы сократить
vladimir.khablak requested changes 2026-03-18 11:30:37 +00:00
@ -0,0 +87,4 @@
return;
}
const fetchTypes = async () => {
const client = new Client();

может тогда лучше сделать client синглтоном прямо в файле api? Чтобы сразу экспортировать и не делать new Client когда нужен запрос?

может тогда лучше сделать client синглтоном прямо в файле api? Чтобы сразу экспортировать и не делать new Client когда нужен запрос?
@ -0,0 +161,4 @@
return;
}
try {
const client = new Client();

идентично

идентично
@ -0,0 +35,4 @@
}) => {
const intl = useIntl();
const selectedTypeName = types.find((t) => String(t.id) === badgeType)?.name ||

мб в мемо?

мб в мемо?
@ -0,0 +38,4 @@
const selectedTypeName = types.find((t) => String(t.id) === badgeType)?.name ||
intl.formatMessage({id: 'badges.modal.field_type_placeholder', defaultMessage: 'Выберите тип достижения'});
const triggerLabel = showCreateType ? intl.formatMessage({id: 'badges.modal.create_new_type', defaultMessage: '+ Создать новый тип'}) : selectedTypeName;
const confirmType = confirmDeleteTypeId ? types.find((t) => String(t.id) === confirmDeleteTypeId) : null;

мб в мемо?

мб в мемо?
@ -0,0 +78,4 @@
// Prefill пользователя, если передан
if (modalData?.prefillUser) {
Client4.getUserByUsername(modalData.prefillUser).then((user) => {

получается что есть api а есть еще какое-то апи, на твое усмотрение можно запихнуть в api плагина

получается что есть api а есть еще какое-то апи, на твое усмотрение можно запихнуть в api плагина
@ -0,0 +108,4 @@
const handleClose = useCallback(() => {
setClosing(true);
setTimeout(doClose, 150);

на твое усмотрение - вынести число в константу

на твое усмотрение - вынести число в константу
@ -37,2 +43,2 @@
};
}
const fetchBadges = useCallback(() => {
const client = new Client();

идентично

идентично
@ -0,0 +45,4 @@
const handleEdit = useCallback((badgeType: BadgeTypeDefinition) => {
dispatch(openEditTypeModal(badgeType));
}, [dispatch]);

как будто и не надо в зависимости добавлять

как будто и не надо в зависимости добавлять
@ -45,1 +128,3 @@
case RHS_STATE_DETAIL:
};
const renderContent = () => {

выглядит как отдельный компонент или useMemo

выглядит как отдельный компонент или useMemo
@ -87,0 +90,4 @@
if (!this.props.user) {
return;
}
const c = new Client();

пупупуууу

пупупуууу
@ -0,0 +78,4 @@
setLoading(true);
setError(null);
try {
const client = new Client();

опять клиент

опять клиент
@ -0,0 +110,4 @@
setLoading(true);
setError(null);
try {
const client = new Client();

опять он

опять он
@ -0,0 +41,4 @@
useEffect(() => {
let cancelled = false;
if (loadedValueRef.current === value) {

как будто можно инвертировать условие if (loadedValueRef.current !== value && value) {...} или типо того

как будто можно инвертировать условие if (loadedValueRef.current !== value && value) {...} или типо того
@ -0,0 +165,4 @@
) : (
<SearchIcon/>
)}
{profilesLoading ? null : selectedUsers.map((user) => (

на твое усмотрение {!profilesLoading && selectedUsers.map...}

на твое усмотрение {!profilesLoading && selectedUsers.map...}
@ -49,3 +42,2 @@
componentDidMount() {
useEffect(() => {
const c = new Client();

client

client
@ -75,2 +57,4 @@
return;
}
const toShow = groups.slice(0, MAX_BADGES);
const names = toShow.

как будто filter.map.filter можно заменить на reduce

как будто filter.map.filter можно заменить на reduce
@ -159,1 +105,3 @@
onClick={this.onMoreClick}
return (
<div id='badgePlugin'>
<div><b>

немного поплыли стили

немного поплыли стили
@ -0,0 +43,4 @@
return (
<>
{nameRow}{'\n'}

такой конструкции я еще не видел, жестка

такой конструкции я еще не видел, жестка
Owner

Что тут происходит?) Согласен с Владмиром

Что тут происходит?) Согласен с Владмиром
@ -30,6 +30,13 @@ export const getRHSView = createSelector(
},
);
export const getPrevRHSView = createSelector(

разве первым аргументом не должна идти строка? типо 'getPrevRHSView' или что-то такое?

разве первым аргументом не должна идти строка? типо 'getPrevRHSView' или что-то такое?
This pull request can be merged automatically.
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin LP-5613:LP-5613
git checkout LP-5613
Sign in to join this conversation.
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: loop/loop-plugin-achievements#2
No description provided.