Skip to content

Conversation

@felipe-rod123
Copy link
Contributor

Proposed changes (including videos or screenshots)

This pull request helps organize the main functions from apps/meteor/client/views/marketplace/helpers.ts, by creating individual files in the apps/meteor/client/views/marketplace/helpers/ folder.

Issue(s)

Steps to test or reproduce

Further comments

MKP-167

@codecov
Copy link

codecov bot commented Mar 25, 2023

Codecov Report

Merging #28611 (26947ed) into develop (5408835) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28611      +/-   ##
===========================================
- Coverage    46.67%   46.60%   -0.07%     
===========================================
  Files          699      699              
  Lines        13081    13081              
  Branches      2228     2228              
===========================================
- Hits          6106     6097       -9     
- Misses        6656     6663       +7     
- Partials       319      321       +2     
Flag Coverage Δ
e2e 46.57% <ø> (-0.07%) ⬇️

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

@felipe-rod123 felipe-rod123 requested a review from rique223 March 26, 2023 14:52
@felipe-rod123 felipe-rod123 marked this pull request as ready for review March 26, 2023 14:52
@felipe-rod123 felipe-rod123 requested a review from a team as a code owner March 26, 2023 14:52
rique223
rique223 previously approved these changes Mar 27, 2023
Copy link
Contributor

@rique223 rique223 left a comment

Choose a reason for hiding this comment

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

LGTM!

@rique223 rique223 added this to the 6.1.0 milestone Mar 27, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Mar 27, 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 27, 2023
@casalsgh casalsgh modified the milestones: 6.1.0, 6.1.0 special Mar 27, 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 27, 2023
@dionisio-bot dionisio-bot bot removed the stat: ready to merge PR tested and approved waiting for merge label Mar 27, 2023
@d-gubert d-gubert removed this from the 6.1.0 special milestone Mar 27, 2023
@tiagoevanp tiagoevanp added this to the 6.3.0 milestone Apr 19, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: conflict labels Jun 7, 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 Jun 7, 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 Jun 7, 2023
@felipe-rod123 felipe-rod123 requested a review from rique223 June 7, 2023 17:00
@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 Jun 7, 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 Jun 7, 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 Jun 9, 2023
@kodiakhq kodiakhq bot merged commit b31ccd4 into develop Jun 9, 2023
@kodiakhq kodiakhq bot deleted the chore/new-helper-functions-files branch June 9, 2023 15:05
gabriellsh added a commit that referenced this pull request Jun 12, 2023
…memberList

* 'develop' of github.com:RocketChat/Rocket.Chat: (43 commits)
  chore: Pages empty state consistency (#29439)
  refactor: Convert Omnichannel helper ee to ts (#29274)
  regression(i18n): Broken translations on client-side startup (#29442)
  chore: break down helpers.ts and create new files (#28611)
  test: runInBand unit test (#29484)
  i18n: system message copy updates (#29372)
  chore: emoji popup trigger length (#29282)
  fix: Seat counter including apps (#29489)
  fix: Invalid message draft (#29474)
  fix: message deletion fails if has files attached on filesystem storage (#29313)
  chore: replace meteor.call -> sdk.call (#29318)
  chore: deprecation api (#29347)
  fix: Imported messages are not displayed (#29416)
  chore: convert metrics to ts (#29475)
  chore: use new Livechat SDK Implementation (#29098)
  fix: Room history scroll position (#29335)
  fix: check for $addToSet to be not empty before passing update params (#29378)
  chore: update `badge-level-0` color (#29460)
  fix: Saving Business hour throws an alert (#29449)
  regression: emojiPicker position (#29408)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants