Skip to content
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.

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

Review legacy Security PATTERNS for migration #4036

Closed
Tracked by #4611
elycheea opened this issue Jan 10, 2024 · 1 comment
Closed
Tracked by #4611

Review legacy Security PATTERNS for migration #4036

elycheea opened this issue Jan 10, 2024 · 1 comment
Assignees
Labels
area: migration ➡️ Migration of Security package to IBM Products package: security

Comments

@elycheea
Copy link
Contributor

elycheea commented Jan 10, 2024

Related issues:

Patterns (as Components)

These are Security patterns that are still widely used and DO NOT have a competing component in Carbon or Carbon for IBM Products.

This is a summary report. The details listed for each component is not comprehensive.


FilterPanel

GitHub, Storybook

FilterPanel comes with 8 subcomponents, including search, panel, group, label, accordion (2), and checkbox (2).

With a design review, these can be expanded to include more features such as radio buttons, date pickers, sliders, etc.

FilterPanel will easily be the largest and most complex component to migrate.

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS (includes all subcomponents)

import { Accordion } from '../../Accordion';
import { AccordionItem } from '../../Accordion';
import Checkbox from '../../Checkbox';
import OverflowMenu from '../../OverflowMenu';
import TruncatedList from '../../TruncatedList';

IconButtonBar

GitHub, Storybook

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS

import { OverflowMenu, OverflowMenuItem } from '../..';
import IconButton from '../IconButton';

NonEntitledSection

GitHub, Storybook

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS

import Link from '../Link';

SearchBar

GitHub, Storybook

TLDR

  • See This component must be refactored from a Class to a Function. (Link)

PROBLEMATIC IMPORTS

import Button from '../Button';
import Search from '../Search';
import MultiSelect from '../MultiSelect';

StatusIndicator

GitHub, Storybook

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS

import Button from '../Button';

TagWall

GitHub, Storybook

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS

import Button from '../Button';
import InteractiveTag from '../InteractiveTag';

PROBLEMATIC CODE

const { TAG_WALL_ADD_BUTTON, TAG_WALL_LABEL, TAG_WALL_REMOVE_BUTTON } = {
  ...defaultLabels.labels,
  ...defaultLabels.filterFalsey({
    TAG_WALL_ADD_BUTTON: addLabel,
    TAG_WALL_LABEL: label,
  }),
  ...labels,
};

TagWallFilter

GitHub, Storybook

Includes one subcomponent: filter

TLDR

  • Follows Carbon for IBM Products component architecture.

PROBLEMATIC IMPORTS

import { TearsheetSmall } from '../Tearsheet';
import { buttonType } from '../Tearsheet/TearsheetSmall/TearsheetSmall';
@elycheea elycheea converted this from a draft issue Jan 10, 2024
@paul-balchin-ibm paul-balchin-ibm moved this from Backlog 🌋 to In progress in Carbon for IBM Products Jan 11, 2024
@paul-balchin-ibm paul-balchin-ibm changed the title Review legacy Security patterns for migration Review legacy Security PATTERNS for migration Jan 15, 2024
@elycheea elycheea moved this from In progress to In review 👀 in Carbon for IBM Products Jan 22, 2024
@elycheea elycheea mentioned this issue Jan 24, 2024
5 tasks
@elycheea elycheea added the area: migration ➡️ Migration of Security package to IBM Products label Jan 24, 2024
@elycheea
Copy link
Contributor Author

elycheea commented Apr 2, 2024

Closing for now since it’s documented as part of the Security migration epic.

@elycheea elycheea closed this as completed Apr 2, 2024
@github-project-automation github-project-automation bot moved this from In review 👀 to Done 🚀 in Carbon for IBM Products Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: migration ➡️ Migration of Security package to IBM Products package: security
Projects
Archived in project
Development

No branches or pull requests

2 participants