Add storybook tests on ObjectFilterDropdownOptionSelect

[thomtrp]

Scope & Context

Follow-up on this PR: #3656
New components have been added without any storybook test.

Technical inputs

File paths are the following:

• packages/twenty-front/src/modules/ui/navigation/menu-item/components/MenuItemSelectTag.tsx
• packages/twenty-front/src/modules/object-record/object-filter-dropdown/components/ObjectFilterDropdownOptionSelect.tsx

[anoopw3bdev]

Hi @thomtrp I took a look the folders, none of the components in packages/twenty-front/src/modules/object-record/object-filter-dropdown/components have a story, ideally when we create a new component we should write the respective story too?

[ZichenMen]

I'm going to work on this issue.

[clrminded]

I will be working on this issue as well.

[BalanaguYashwanth]

Hello @thomtrp,

• MenuItemSelectTag - already story exists for this component

• ObjectFilterDropdownOptionSelect - related to object-filter-dropdown folder, there are no stories written in it.

• So can you close this issue and create separate issue to write stories for object-filter-dropdown folder

[charlesBochet]

@BalanaguYashwanth Thanks for checking, I have renamed the issue

[Bonapara]

/oss.gg 150

[oss-gg]

Thanks for opening an issue! It's live on oss.gg!

[priyanshu-kun]

/assign

[oss-gg]

Assigned to @priyanshu-kun! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[pra2107tham]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[yashdarji30]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[oss-gg]

@priyanshu-kun, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[sateshcharan]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[oss-gg]

@priyanshu-kun, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[charlesBochet]

/unassign

[oss-gg]

Issue unassigned.

[rashap224]

/assign

[oss-gg]

Assigned to @rashap224! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[rashap224]

Hi @thomtrp Can you explain a bit about this issue. I am curious to know and work on it.

[oss-gg]

@priyanshu-kun, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[oss-gg]

@rashap224, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

[charlesBochet]

/unassign

[oss-gg]

Issue unassigned.

[Khaan25]

Hey @charlesBochet, Can I work on this?

[charlesBochet]

sure

[Khaan25]

Hey @charlesBochet, I've created story and only imported the component to see but I'm getting this error? Do we've some providers to wrap around the story in decorators?

[FelixMalfait]

From @charlesBochet on the PR:

I have taken a close look, looks like the ObjectFilterDropdownOptionSelect component is coupled with many things: 4 recoil states + parent dropdown + selectableList + object filter dropdown. Which is typical from an internal component that is not meant to be used of its context.

Because of that, I don't think it makes a lot of sense to test it as a standalone.

Instead, we should add more cases (including the Select field type here) to multiplefiltersdropdownbutton story