Skip to content

Conversation

@mocca102
Copy link
Contributor

No description provided.

@mocca102 mocca102 requested a review from a team December 19, 2023 17:42
Copy link
Contributor

@esezen esezen left a comment

Choose a reason for hiding this comment

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

This is looking good 🚀 I think we should revert the type changes, let me know if you disagree

import ConstructorIOClient from '@constructor-io/constructorio-client-javascript';
import { CioAutocomplete } from '../../../index';
import { CioAutocomplete, CioAutocompleteProps } from '../../../index';
import { argTypes } from '../argTypes';
Copy link
Contributor

Choose a reason for hiding this comment

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

Lint

src/types.ts Outdated
export type Section = AutocompleteSection | RecommendationsSection | CustomSection;

export type UserDefinedSection = CustomSection | SectionConfiguration;
export type UserDefinedSection = Section;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we changing types on a Storybook PR? I think this is fixed in another PR already so maybe we can skip it here?

@mocca102 mocca102 requested a review from esezen December 26, 2023 19:46
Copy link
Contributor

@esezen esezen left a comment

Choose a reason for hiding this comment

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

LGTM!

@esezen esezen merged commit b386220 into main Dec 29, 2023
@esezen esezen deleted the csl-2817-autocomplete-os-ui-better-storybook-version branch December 29, 2023 18:15
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