Skip to content

Chore: remove unused prop in register workspace modal#28317

Merged
kodiakhq[bot] merged 4 commits into
developfrom
fix/remove-unused-prop-register-workspace
Mar 10, 2023
Merged

Chore: remove unused prop in register workspace modal#28317
kodiakhq[bot] merged 4 commits into
developfrom
fix/remove-unused-prop-register-workspace

Conversation

@csuadev

@csuadev csuadev commented Mar 7, 2023

Copy link
Copy Markdown
Contributor

Remove prop that wasn't needed at the end in the registering workspace modal.

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@csuadev csuadev requested a review from a team as a code owner March 7, 2023 19:16
@codecov

codecov Bot commented Mar 8, 2023

Copy link
Copy Markdown

Codecov Report

Merging #28317 (914163a) into develop (67ba590) will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28317      +/-   ##
===========================================
+ Coverage    45.08%   45.21%   +0.12%     
===========================================
  Files          759      759              
  Lines        14784    14784              
  Branches      2085     2085              
===========================================
+ Hits          6665     6684      +19     
+ Misses        7824     7808      -16     
+ Partials       295      292       -3     
Flag Coverage Δ
e2e 45.17% <ø> (+0.12%) ⬆️

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

@csuadev csuadev requested a review from jcbrtl March 9, 2023 14:41

@jcbrtl jcbrtl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is just a cleanup but I'm kinda paranoid at this stage, so we double checked everything anyway (thanks @janainaCoelhoRocketchat ) 👍

@yash-rajpal yash-rajpal added the stat: ready to merge PR tested and approved waiting for merge label Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@kodiakhq kodiakhq Bot merged commit 1159cb3 into develop Mar 10, 2023
@kodiakhq kodiakhq Bot deleted the fix/remove-unused-prop-register-workspace branch March 10, 2023 14:56
gabriellsh added a commit that referenced this pull request Mar 13, 2023
…ketChat/Rocket.Chat into matrixSearchTestBranch

* 'feat/federation-public-room-search-2' of github.com:RocketChat/Rocket.Chat: (42 commits)
  refactor: Replace `_.isObject` with `isObject` (#28376)
  fix: small tweaks
  chore: Remove old css (#28140)
  fix: Thread list not scrolling to bottom when sending a message. (#28346)
  refactor: Federation refactor towards services + file names improvement + E2E tests fixes (#28326)
  i18n: Language update from LingoHub 🤖 on 2023-03-10Z (#28365)
  refactor: Improve sidebar search list hook (#28328)
  Chore: Change the PR title check to use conventional commits (#28358)
  [IMPROVE] Galician onboarding translation (#27908)
  [IMPROVE] Adding input type=password to AWS secrets fields (#28159)
  [FIX] Normal and multiline messages adding extra spaces upon copying/pasting (#28187)
  Chore: remove unused prop in register workspace modal (#28317)
  [FIX] App crashing when switching to Hindi (hi-In) language (#28285)
  [NEW] Registered Loom oEmbed (#28339)
  fix: create dm users before join the room
  tests: add unit tests + small tweaks
  fix: fix invite links button logic
  chore: fix lint
  Chore: Remove/replace some Meteor packages (#28189)
  Chore: Unused/dead code removal (#28155)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants