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

feat(api-client,app,react-api-client): upload splash logo from desktop app #14941

Merged
merged 10 commits into from
Apr 22, 2024

Conversation

brenthagen
Copy link
Contributor

@brenthagen brenthagen commented Apr 17, 2024

Overview

adds the upload input component, api-client, and react-api-client functions needed to upload a splash logo from the factory mode slideout

closes PLAT-283

Test Plan

verified successful logo upload 201. further tests will be added in PLAT-292

Changelog

  • Adds functions to upload from the factory mode slideout

Review requests

check the upload

Risk assessment

low

…p app

adds the upload input component, api-client, and react-api-client functions needed to upload a
splash logo from the factory mode slideout

PLAT-283
@brenthagen brenthagen requested a review from vegano1 April 17, 2024 20:38
Copy link
Contributor

@vegano1 vegano1 left a comment

Choose a reason for hiding this comment

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

Awesome,
Just left a comment for later
Otherwise looking good.

isUploading ||
fileError != null
}
onClick={handleCompleteClick}
Copy link
Contributor

Choose a reason for hiding this comment

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

Eventually, It would be good to have a confirmation modal here, since the robot will be restarted.

@brenthagen brenthagen marked this pull request as ready for review April 22, 2024 15:00
@brenthagen brenthagen requested a review from a team as a code owner April 22, 2024 15:00
@brenthagen brenthagen requested review from jerader and a team and removed request for a team and jerader April 22, 2024 15:00
@brenthagen brenthagen merged commit 433ef44 into edge Apr 22, 2024
50 checks passed
@brenthagen brenthagen deleted the app_splash-logo-upload branch April 22, 2024 20:59
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
…p app (#14941)

adds the upload input component, api-client, and react-api-client
functions needed to upload a splash logo from the factory mode slideout

closes PLAT-283
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.

None yet

2 participants