Skip to content

Release changes - UI Admin & Backend Refactor#35

Merged
misha130 merged 11 commits into
developmentfrom
feat/ui-changes
Jul 20, 2026
Merged

Release changes - UI Admin & Backend Refactor#35
misha130 merged 11 commits into
developmentfrom
feat/ui-changes

Conversation

@misha130

Copy link
Copy Markdown
Contributor
image

@misha130 misha130 changed the title UI Refactor Release changes - UI Admin & Backend Refactor Jul 20, 2026
Comment thread core/src/Application/Admin/ConfigReleases/Commands/PublishConfigReleaseCommand.cs Outdated
Comment thread core/src/Application/Common/ConfigEntryContentTypes.cs Outdated
@misha130
misha130 requested a review from SonyStone July 20, 2026 16:38

@SonyStone SonyStone left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There are a lot of large components. There's nothing wrong with that in itself, but it's better to spread the logic across different services.

Comment thread admin/src/app/App.tsx Outdated
Comment thread admin/src/app/App.tsx Outdated
Comment thread admin/src/entities/project/api/metadata.service.ts
Comment thread admin/src/entities/project/model/active-project.ts Outdated
Comment thread admin/src/entities/project/model/active-environment.ts Outdated
import { ProjectsStats } from "./components/ProjectsStats";

export default function ProjectsPage() {
const location = useLocation();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There's a lot of logic in the component as well; it would also be a good idea to split it into separate services.

}
}}
class={`focus-visible:ring-primary/40 h-full cursor-pointer rounded-lg border p-6 transition-all duration-300 select-none focus-visible:ring-2 focus-visible:outline-none active:scale-[0.98] ${
class={`focus-visible:ring-primary/40 h-full cursor-pointer rounded-lg border p-4 transition-all duration-300 select-none focus-visible:ring-2 focus-visible:outline-none active:scale-[0.98] sm:p-6 ${

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use cn() utility here too

Comment thread admin/src/pages/users/components/UsersFilters.tsx Outdated
Comment thread admin/src/pages/users/components/UsersTable.tsx Outdated
Comment thread admin/src/shared/hooks/useTheme.tsx
@misha130
misha130 merged commit 4ee06ac into development Jul 20, 2026
4 checks passed
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.

3 participants