Skip to content

chore(site): remove template version editor xservice #10490

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 24 commits into from
Nov 3, 2023
Merged

chore(site): remove template version editor xservice #10490

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
c290855
Refactor initial loading
BrunoQuaresma Oct 31, 2023
93df525
Remove non used events
BrunoQuaresma Oct 31, 2023
b3c712c
Move cancel build to create build service
BrunoQuaresma Oct 31, 2023
10d6b4e
Extract upload
BrunoQuaresma Oct 31, 2023
7eb5c91
Extract create version
BrunoQuaresma Nov 1, 2023
c024a1d
Extract logs
BrunoQuaresma Nov 1, 2023
e7713fe
Refactor version fetching and refetch
BrunoQuaresma Nov 1, 2023
f782c7d
Extract resources
BrunoQuaresma Nov 1, 2023
8bf23af
Extract missing variables
BrunoQuaresma Nov 1, 2023
5502077
Extract publish from the machine and remove it
BrunoQuaresma Nov 1, 2023
20d6100
Remove unused data module
BrunoQuaresma Nov 1, 2023
9d1ca61
Fix loading state and tabs code to make it clear
BrunoQuaresma Nov 1, 2023
c73f976
Fix disable preview state
BrunoQuaresma Nov 1, 2023
86735c9
Optimistic update logs
BrunoQuaresma Nov 1, 2023
86d7a6d
Encapsulate a few logics in hooks
BrunoQuaresma Nov 1, 2023
e748fe8
Extract start/end build logic
BrunoQuaresma Nov 1, 2023
71e8920
Fix tests
BrunoQuaresma Nov 2, 2023
8a7fae6
Remove unused comment
BrunoQuaresma Nov 2, 2023
2769cfe
Show initialize error on snackbar
BrunoQuaresma Nov 2, 2023
4893156
Update tarFile in a single state call
BrunoQuaresma Nov 2, 2023
32934ed
Simplify onDone call
BrunoQuaresma Nov 2, 2023
209e364
Make tar manipulation more clear
BrunoQuaresma Nov 2, 2023
66ea796
Improve publish promise construction
BrunoQuaresma Nov 2, 2023
353f325
Fix storybook
BrunoQuaresma Nov 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix storybook
  • Loading branch information
@BrunoQuaresma
BrunoQuaresma committed Nov 2, 2023
commit 353f32522fb6f3c42b810bd5f5a73e2be2a177f7
7 changes: 5 additions & 2 deletions site/src/pages/TemplateVersionEditorPage/TemplateVersionEditor.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,9 @@ type Story = StoryObj<typeof TemplateVersionEditor>;

export const Example: Story = {};

export const Logs = {
export const Logs: Story = {
args: {
defaultTab: "logs",
buildLogs: MockWorkspaceBuildLogs,
templateVersion: {
...MockTemplateVersion,
Expand All @@ -46,6 +47,7 @@ export const Logs = {

export const Resources: Story = {
args: {
defaultTab: "resources",
buildLogs: MockWorkspaceBuildLogs,
resources: [
MockWorkspaceResource,
Expand All @@ -58,8 +60,9 @@ export const Resources: Story = {
},
};

export const ManyLogs = {
export const WithError = {
args: {
defaultTab: "logs",
templateVersion: {
...MockTemplateVersion,
job: {
Expand Down
7 changes: 4 additions & 3 deletions site/src/pages/TemplateVersionEditorPage/TemplateVersionEditor.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ import AlertTitle from "@mui/material/AlertTitle";
import { DashboardFullPage } from "components/Dashboard/DashboardLayout";
import { type Interpolation, type Theme, useTheme } from "@emotion/react";

type Tab = "logs" | "resources" | undefined; // Undefined is to hide the tab
export interface TemplateVersionEditorProps {
template: Template;
templateVersion: TemplateVersion;
Expand All @@ -70,6 +71,7 @@ export interface TemplateVersionEditorProps {
missingVariables?: TemplateVersionVariable[];
onSubmitMissingVariableValues: (values: VariableValue[]) => void;
onCancelSubmitMissingVariableValues: () => void;
defaultTab?: Tab;
}

const topbarHeight = 80;
Expand Down Expand Up @@ -107,11 +109,10 @@ export const TemplateVersionEditor: FC<TemplateVersionEditorProps> = ({
missingVariables,
onSubmitMissingVariableValues,
onCancelSubmitMissingVariableValues,
defaultTab,
}) => {
const theme = useTheme();
const [selectedTab, setSelectedTab] = useState<
"logs" | "resources" | undefined // Undefined is to hide the tab
>();
const [selectedTab, setSelectedTab] = useState<Tab>(defaultTab);
const [fileTree, setFileTree] = useState(defaultFileTree);
const [createFileOpen, setCreateFileOpen] = useState(false);
const [deleteFileOpen, setDeleteFileOpen] = useState<string>();
Expand Down