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

refactor(components): refactor roundtab stories #14956

Merged
merged 2 commits into from
Apr 22, 2024
Merged

Conversation

koji
Copy link
Contributor

@koji koji commented Apr 19, 2024

update roundtab stories to align with the actual app design

close AUTH-

Overview

update RoundTab component stories to align with the actual app design

2024-04-19 00_43_39-Library _ Molecules _ RoundTab - Round Tab ⋅ Storybook and 85 more pages - Perso

close AUTH-358

Test Plan

Changelog

Review requests

Risk assessment

low

update roundtab stories to align with the actual app design

close AUTH-
@koji koji requested review from ncdiehl11 and jerader April 19, 2024 04:47
@koji koji marked this pull request as ready for review April 19, 2024 04:47
@koji koji requested a review from a team as a code owner April 19, 2024 04:47
@@ -1,55 +1,80 @@
import * as React from 'react'
import React from 'react'
Copy link
Collaborator

Choose a reason for hiding this comment

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

why did you have to change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

and slightly smaller than * as React. (I'm using React for opentrons-ai-client)
I will use * as React for this moment.

Copy link
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

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

lgtm! left one comment

@koji koji merged commit ec73d82 into edge Apr 22, 2024
48 checks passed
@koji koji deleted the refactor_round-tab-stories branch April 22, 2024 17:01
y3rsh added a commit that referenced this pull request Apr 23, 2024
* edge: (194 commits)
  fix(app): clone run with RTPs from HistoricalProtocolRun (#14959)
  fix(api): Filter out `air_gap()` calls as higher-order commands (#14985)
  fix(app): fix infinitely re-rendering/never rendering firmware success toasts (#14981)
  feat(api): add option to ignore different tip presence states (#14980)
  feat(opentrons-ai-client) add input textbox to container (#14968)
  fix(app): add robotSerialNumber to proceedToRun event (#14976)
  fix(api): remove homing patch fix for right mount when a 96-channel is attached (#14975)
  feat(api-client,app,react-api-client): upload splash logo from desktop app (#14941)
  fix(robot-server): notify /runs when a non-current run is deleted (#14974)
  feature(api, robot-server): Allow fixit commands to recover from an error (#14908)
  feat(hardware-testing): enable multi sensor processing in liquid probe (#14883)
  fix(app): prevent "run again" banner from rendering after navigating away from the current run (#14973)
  refactor(components): refactor roundtab stories (#14956)
  refactor(protocol-designer): assign module slot in createFileWizard instead of modal (#14951)
  fix(app, api-client): fix choose protocol slideout issue (#14949)
  refactor(protocol-designer): tip position modal max values round down (#14972)
  feat(app): add tiprack selection step to quick transfer flow (#14950)
  ci(shared-data): install dependencies in workflow (#14958)
  fix(components): fix icon stories (#14969)
  feat(opentrons-ai-client): introduce react-markdown to chat display component (#14965)
  ...
Carlos-fernandez pushed a commit that referenced this pull request May 20, 2024
* refactor(components): refactor roundtab stories
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants