Skip to content
Snippets Groups Projects
  1. Jun 09, 2022
    • Tiago Evangelista Pinto's avatar
      [FIX] AgentsPage pagination (#25820) · 83851a0a
      Tiago Evangelista Pinto authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      83851a0a
    • Murtaza Patrawala's avatar
      Chore: Move voip's Wrap-up and On-hold functionality to EE (Backend) (#25160) · a741aecd
      Murtaza Patrawala authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      a741aecd
    • Murtaza Patrawala's avatar
      [FIX] Access issue on chat.getThreadsList (#25750) · 9f2c03c2
      Murtaza Patrawala authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      Clickup task: https://app.clickup.com/t/201cbrc
      9f2c03c2
    • Murtaza Patrawala's avatar
      Chore: Remove snap files from Houston config (#25819) · 92a25795
      Murtaza Patrawala authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      Since these snap files were recently removed [here](https://github.com/RocketChat/Rocket.Chat/pull/25790), we'd need to update out release CLI accordingly 
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      92a25795
    • Murtaza Patrawala's avatar
      [FIX] Voip endpoint permissions (#25783) · 5a74a5c1
      Murtaza Patrawala authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      Earlier we didn't check for any permissions while creating or closing VoIP room. This new PR will enforce those permission checks
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      5a74a5c1
    • Murtaza Patrawala's avatar
      [FIX] allow only livechat-agents to be contact manager for any omnichannel contact (#25451) · 6b3908bf
      Murtaza Patrawala authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      ![image](https://user-images.githubusercontent.com/34130764/169371479-30b0b88b-08e8-4975-8f36-6387d968a568.png)
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      
      Tasks:
      
      - [x] Frontend change on auto-select component
      - [x] Backend API checks
      6b3908bf
    • Kevin Aleman's avatar
      Chore: use params instead of URL building on livechat endpoints (#25810) · 11a9d235
      Kevin Aleman authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      11a9d235
    • Guilherme Gazzo's avatar
      Regression: fix apps path (#25809) · c27412b0
      Guilherme Gazzo authored
      c27412b0
    • Kevin Aleman's avatar
      [BREAK] Remove RDStation integration (#25774) · 993745d3
      Kevin Aleman authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
      
      
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      
      
      Co-authored-by: default avatarGuilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
      993745d3
  2. Jun 08, 2022
    • Guilherme Gazzo's avatar
      Chore: RestApiClient as Package (#25469) · 611c0b31
      Guilherme Gazzo authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
      
      
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      
      Co-authored-by: default avatarTasso Evangelista <2263066+tassoevan@users.noreply.github.com>
      611c0b31
    • Duda Nogueira's avatar
    • Tiago Evangelista Pinto's avatar
      [FIX] AccountBox checks for condition (#25708) · 2a292e4f
      Tiago Evangelista Pinto authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      Close: #25704
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      Fixes #25704 
      2a292e4f
    • Guilherme Gazzo's avatar
      Chore: Fix CI (#25797) · 9629831b
      Guilherme Gazzo authored
      9629831b
    • Kevin Aleman's avatar
      [FIX] Fix prom-client new promise usage (#25781) · b021a2fe
      Kevin Aleman authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      <!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      Looks like the update of de dep had breaking changes and now uses promises at some calls. This PR fixes those promise usages and converts the file to TS (hopefully it helps us to spot the error in the future)
      b021a2fe
    • Hugo Costa's avatar
      [FIX] Discussion alphabetical ordering (#25788) · 4391ca64
      Hugo Costa authored
      <!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
      
      <!-- Your Pull Request name should start with one of the following tags
        [NEW] For new features
        [IMPROVE] For an improvement (performance or little improvements) in existing features
        [FIX] For bug fixes that affect the end-user
        [BREAK] For pull requests including breaking changes
        Chore: For small tasks
        Doc: For documentation
      -->
      
      <!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. 
        - I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
        - I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
        - Lint and unit tests pass locally with my changes
        - I have added tests that prove my fix is effective or that my feature works (if applicable)
        - I have added necessary documentation (if applicable)
        - Any dependent changes have been merged and published in downstream modules
      -->
      
      ## Proposed changes (including videos or screenshots)
      <!-- CHANGELOG -->
      <!--
        Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
        If it fixes a bug or resolves a feature request, be sure to link to that issue below.
        This description will appear in the release notes if we accept the contribution.
      -->
      Added a validation in the prop used for sorting (loweCaseName) checking for a prop that only exists in discussions (prid)
      
      <!-- END CHANGELOG -->
      
      ## Issue(s)
      <!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
      
      ## Steps to test or reproduce
      Steps to reproduce:
      
      1. Log in to the [rocket.chat](http://rocket.chat/) (version 4.8.0-rc.1)
      2. Click on the button Display and select the option Sort by : Name 
      
      Actual result:
      
      -   The discussions were not sorted by name in all types of Group: Unread, Favorites and Types 
      
      ![image](https://user-images.githubusercontent.com/20212776/172384754-3d33c76b-926b-4814-bb2f-22c7f0f914ae.png)
      
      Expected result:
      The discussions should be sorted by name
      
      ## Further comments
      <!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
      4391ca64
    • Guilherme Gazzo's avatar
      Update .kodiak.toml · ccc869fd
      Guilherme Gazzo authored
      ccc869fd
    • Guilherme Gazzo's avatar
      Update .kodiak.toml · 7ea6491c
      Guilherme Gazzo authored
      7ea6491c
    • Guilherme Gazzo's avatar
      Chore: Testing Kodiak feature (#25794) · e675d526
      Guilherme Gazzo authored
      e675d526
  3. Jun 07, 2022
  4. Jun 06, 2022
  5. Jun 05, 2022
  6. Jun 04, 2022
  7. Jun 03, 2022
Loading