Skip to content

Commit

Permalink
Update protobuf import paths for organization onboarding settings and…
Browse files Browse the repository at this point in the history
… add iam session unit test assertions

- Refactor import types to use the updated OrganizationSettings_OnboardingSettings_WelcomeMessage
- Adjust type references in OrgMemberAvatarInput, WelcomeMessageConfigurationField, and WelcomeMessageEditor
- Add newUser flag checks in IAM session app spec
- Remove unused import in organization service

Tool: gitpod/catfood.gitpod.cloud
  • Loading branch information
filiptronicek committed Feb 7, 2025
1 parent b7353c3 commit b4149c6
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,12 @@
import { useState } from "react";
import { useListOrganizationMembers } from "../../data/organizations/members-query";

import type { OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import type { OrganizationSettings_OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import { Button } from "@podkit/buttons/Button";
import { DropdownMenu, DropdownMenuContent, DropdownMenuItem, DropdownMenuTrigger } from "@podkit/dropdown/DropDown";

type Props = {
settings: OnboardingSettings_WelcomeMessage | undefined;
settings: OrganizationSettings_OnboardingSettings_WelcomeMessage | undefined;
setFeaturedMemberId: (featuredMemberId: string | undefined) => void;
};
export const OrgMemberAvatarInput = ({ settings, setFeaturedMemberId }: Props) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
*/

import { PlainMessage } from "@bufbuild/protobuf";
import type { OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import type { OrganizationSettings_OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import { SwitchInputField } from "@podkit/switch/Switch";
import { Heading3, Subheading } from "@podkit/typography/Headings";
import { useCallback, useState } from "react";
Expand Down Expand Up @@ -40,7 +40,10 @@ export const WelcomeMessageConfigurationField = ({ handleUpdateTeamSettings }: P
const [welcomeMessageEditorOpen, setWelcomeMessageEditorOpen] = useState(false);

const handleUpdateWelcomeMessage = useCallback(
async (newSettings: PlainMessage<OnboardingSettings_WelcomeMessage>, options?: UpdateTeamSettingsOptions) => {
async (
newSettings: PlainMessage<OrganizationSettings_OnboardingSettings_WelcomeMessage>,
options?: UpdateTeamSettingsOptions,
) => {
await handleUpdateTeamSettings(
{
onboardingSettings: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
*/

import { PlainMessage } from "@bufbuild/protobuf";
import type { OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import type { OrganizationSettings_OnboardingSettings_WelcomeMessage } from "@gitpod/public-api/lib/gitpod/v1/organization_pb";
import { Button } from "@podkit/buttons/Button";
import { LoadingButton } from "@podkit/buttons/LoadingButton";
import { Textarea } from "@podkit/forms/TextArea";
Expand All @@ -19,12 +19,12 @@ import { OrgMemberAvatarInput } from "./OrgMemberAvatarInput";
import { gitpodWelcomeSubheading } from "./WelcomeMessageConfigurationField";

type Props = {
settings: OnboardingSettings_WelcomeMessage | undefined;
settings: OrganizationSettings_OnboardingSettings_WelcomeMessage | undefined;
isLoading: boolean;
isOwner: boolean;
isOpen: boolean;
handleUpdateWelcomeMessage: (
newSettings: PlainMessage<OnboardingSettings_WelcomeMessage>,
newSettings: PlainMessage<OrganizationSettings_OnboardingSettings_WelcomeMessage>,
options?: UpdateTeamSettingsOptions,
) => Promise<void>;
setIsOpen: (isOpen: boolean) => void;
Expand Down
2 changes: 2 additions & 0 deletions components/server/src/iam/iam-session-app.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,7 @@ class TestIamSessionApp {

expect(result.statusCode, JSON.stringify(result.body)).to.equal(200);
expect(result.body?.userId).to.equal("id-new-user");
expect(result.body?.newUser).to.be.true;
expect(JSON.stringify(result.get("Set-Cookie"))).to.contain(this.cookieName);
}

Expand All @@ -152,6 +153,7 @@ class TestIamSessionApp {

expect(result.statusCode, JSON.stringify(result.body)).to.equal(200);
expect(result.body?.userId).to.equal("id-known-user");
expect(result.body?.newUser).to.be.false;
expect(JSON.stringify(result.get("Set-Cookie"))).to.contain(this.cookieName);
}

Expand Down
1 change: 0 additions & 1 deletion components/server/src/orgs/organization-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ import { CostCenter_BillingStrategy } from "@gitpod/gitpod-protocol/lib/usage";
import { CreateUserParams, UserAuthentication } from "../user/user-authentication";
import isURL from "validator/lib/isURL";
import { DBTeamMembership } from "@gitpod/gitpod-db/lib/typeorm/entity/db-team-membership";
import { TrustedValue } from "@gitpod/gitpod-protocol/lib/util/scrubbing";

@injectable()
export class OrganizationService {
Expand Down

0 comments on commit b4149c6

Please sign in to comment.