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

Revert "feat(app-shell-odd): create node layer for ODD" #11941

Merged
merged 1 commit into from
Jan 3, 2023

Conversation

shlokamin
Copy link
Member

@shlokamin shlokamin commented Jan 3, 2023

Reverts #11852

Overview

Electron rebuild was not able to build the app successfully, reverting for now and will fix the issue in a new PR

@shlokamin shlokamin requested review from a team as code owners January 3, 2023 22:27
@shlokamin shlokamin requested review from koji and removed request for a team January 3, 2023 22:27
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #11941 (0486e5c) into edge (9422935) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #11941      +/-   ##
==========================================
- Coverage   74.22%   74.21%   -0.01%     
==========================================
  Files        1431     1431              
  Lines       47364    47360       -4     
  Branches     3013     3013              
==========================================
- Hits        35154    35150       -4     
  Misses      11743    11743              
  Partials      467      467              
Flag Coverage Δ
app 61.07% <0.00%> (-0.14%) ⬇️
components 65.28% <ø> (ø)
g-code-testing 97.19% <ø> (ø)
hardware 56.89% <ø> (ø)
hardware-testing ∅ <ø> (∅)
labware-library 49.74% <ø> (ø)
notify-server 89.13% <ø> (ø)
protocol-designer 45.89% <ø> (ø)
react-api-client 83.05% <ø> (ø)
shared-data 85.47% <ø> (ø)
step-generation 88.46% <ø> (ø)
update-server 65.47% <ø> (ø)
usb-bridge 81.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
app-shell/src/buildroot/index.ts 0.00% <0.00%> (ø)
app-shell/src/config/migrate.ts 93.33% <ø> (-0.55%) ⬇️

@shlokamin shlokamin merged commit 30d4c05 into edge Jan 3, 2023
@shlokamin shlokamin deleted the revert-11852-app-shell_create-node-layer-for-ODD branch January 3, 2023 22:39
y3rsh added a commit that referenced this pull request Jan 6, 2023
* origin/edge: (34 commits)
  refactor(app): update desktop robot settings calibration section for OT-3 (#11942)
  feat(hardware): add CAN message to update motor position from encoders (#11868)
  ci(monorepo): upgrade windows versions on github workflows (#11940)
  feat(app): implement useCalibrationTaskList hook (#11894)
  feat(app, app-shell, app-shell-odd): create node layer for ODD (#11944)
  refactor(app): Remove recalibrate option from POC and TLC overflow menus [RAUT-93] (#11915)
  fix(api): home z should home gripper z too (#11950)
  refactor(shared-data): gripper use force polynomial function  (#11946)
  refactor(api): move `ModuleGeometry` to legacy protocol core module (#11939)
  refactor(api): deprecate `ModuleContext.geometry` (#11938)
  chore(usb-bridge): add usb-bridge tests to test-py, add restart to push-ot3 (#11937)
  Revert "feat(app-shell-odd): create node layer for ODD (#11852)" (#11941)
  feat(app-shell-odd): create node layer for ODD (#11852)
  feature(hardware): add a warning style to can_mon and an "estop_released" error id (#11924)
  fix(hardware): Remove while loop and rely on  number_of_messages when parsing motor position response.  (#11929)
  fix(hardware): save can_comm / can_mon logs to read-write location (#11933)
  feat(api): Support 96 channel in the hardware controller (#11866)
  refactor(app): revert run a protocol from devices pages (#11909)
  refactor(app): remove warnings (#11922)
  refactor(app): remove invalid type warnings for strings from atoms (#11918)
  ...
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