LP-5613 #2
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "LP-5613"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -66,0 +170,4 @@});}private doDelete = async (url: string, headers: {[x:string]: string} = {}) => {По сути код в методах для crud дублиркется. Можно было бы сократить
@ -0,0 +87,4 @@return;}const fetchTypes = async () => {const client = 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 плагина
@ -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
@ -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) {...} или типо того
@ -0,0 +165,4 @@) : (<SearchIcon/>)}{profilesLoading ? null : selectedUsers.map((user) => (на твое усмотрение {!profilesLoading && selectedUsers.map...}
@ -49,3 +42,2 @@componentDidMount() {useEffect(() => {const c = new Client();client
@ -75,2 +57,4 @@return;}const toShow = groups.slice(0, MAX_BADGES);const names = toShow.как будто filter.map.filter можно заменить на reduce
@ -159,1 +105,3 @@onClick={this.onMoreClick}return (<div id='badgePlugin'><div><b>немного поплыли стили
@ -0,0 +43,4 @@return (<>{nameRow}{'\n'}такой конструкции я еще не видел, жестка
Что тут происходит?) Согласен с Владмиром
@ -30,6 +30,13 @@ export const getRHSView = createSelector(},);export const getPrevRHSView = createSelector(разве первым аргументом не должна идти строка? типо 'getPrevRHSView' или что-то такое?
Checkout
From your project repository, check out a new branch and test the changes.