Skip to content
Snippets Groups Projects
  1. Jun 13, 2022
    • Douglas Fabris's avatar
      Chore: Rewrite RoomWithData (#25858) · 83ec2f90
      Douglas Fabris 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... -->
      Unverified
      83ec2f90
    • souzaramon's avatar
      Chore: remove unused locators from e2e tests (#25860) · 666740ad
      souzaramon authored
      ## Proposed changes (including videos or screenshots)
      - Remove unused locators
      Unverified
      666740ad
    • Martin Schoeler's avatar
      [FIX] fixes HTML sanitizing error. (#25410) · f8065bf4
      Martin Schoeler 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.
      -->
      If the user sent a HTML message over our product to a livechat user the HTML would get rendered on the message box, this prevents it from happening.
      
      <!-- 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 -->
      try to send some html tags in livechat
      
      ## 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... -->
      
      **Will be ready to review as soon as https://github.com/RocketChat/Rocket.Chat/pull/25312
      
       is merged**
      
      Co-authored-by: default avatarCauê Felchar <11652381+cauefcr@users.noreply.github.com>
      Unverified
      f8065bf4
    • Douglas Fabris's avatar
      Chore: Rewrite Admin UsersTable to Typescript (#25698) · 949fcae9
      Douglas Fabris 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>
      Unverified
      949fcae9
    • Felipe's avatar
      Chore: add _id and name options to JSON Schemas (#25813) · e23cd3e0
      Felipe 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.
      -->
      This pull request adds the `roomId` and `roomName` options for the Ajv JSON Schemas on the `packages/rest-typings/src/v1/channels/` and `packages/rest-typings/src/v1/dm/` folders.
      <!-- 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>
      Unverified
      e23cd3e0
    • Pierre Lehnen's avatar
      Chore: Remove unused tokenpass integration code (#25831) · 3c1bc073
      Pierre Lehnen 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... -->
      Unverified
      3c1bc073
    • Diego Sampaio's avatar
    • Yash Rajpal's avatar
      Chore: RouteGroup for My Account sidebar (#25632) · 81388318
      Yash Rajpal 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.
      -->
      Refactoring My Accounts routes. Allows to add "my account" routes for EE.
      <!-- 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 avatarDouglas Fabris <27704687+dougfabris@users.noreply.github.com>
      Unverified
      81388318
    • gabriellsh's avatar
      [FIX] Attachments and OEmbed margins (#25713) · 554091c3
      gabriellsh 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... -->
      Unverified
      554091c3
    • Guilherme Gazzo's avatar
      Chore: Typescript Sidebar RoomList (#25835) · 2a40792c
      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... -->
      Unverified
      2a40792c
    • Guilherme Gazzo's avatar
      Chore: Typescript Sidebar RoomList (#25835) · 79d229bf
      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... -->
      Unverified
      79d229bf
    • Guilherme Gazzo's avatar
      Chore: Typescript Sidebar RoomList (#25835) · 322b6150
      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... -->
      Unverified
      322b6150
    • Ben Wiederhake's avatar
      [FIX] Client-generated sort parameters in channel directory (#25768) · 5a37518e
      Ben Wiederhake authored
      ## Proposed changes (including videos or screenshots)
      - In the web-client, sorting the channel directory by "Last Message" raises the error "Invalid sort parameter provided".
      
      I don't think a screenshot is necessary, but if you'd like one anyway, here it is:
      
      ![Bildschirmfoto_2022-06-06_12-54-34](https://user-images.githubusercontent.com/2690845/172147996-e9942daf-6819-4eee-afa4-b1c6bce7a650.png)
      
      
      ## Issue(s)
      Closes #25695
      
      ## Steps to test or reproduce
      
      - Open the web client, i.e. navigate your browser to `https://rocketchat.$DOMAIN/home`
      - Click the "Directory" button in the top left, (or just navigate directly to `https://rocketchat.$DOMAIN/directory/channels`)
      - In the table header, click on "Last message" once
      - In the table header, click on "Last message" again
      
      Expected behavior: Clicking "Last message" turns on and then toggles sorting by the date of the last message, either first ascending and then descending, or the other way around.
      
      Actual behavior: The first click sorts the messages in ascending order (good!), the second click shows a red warning box "FIXME", the table content disappears, and is replaced by throbbing grey placeholders.
      
      ### "Good" request (ascending order):
      
      `https://rocketchat.domain.org/api/v1/directory?query=%7B%22type%22%3A%22channels%22%2C%22text%22%3A%22%22%2C%22workspace%22%3A%22local%22%7D&sort=%7B%22lastMessage%22%3A1%7D&count=25`
      
      More easily readable GET parameters:
      
      ```
      query | {"type":"channels","text":"","workspace":"local"}
      sort | {"lastMessage":1}
      count | 25
      ```
      
      Response:
      ```
      {"result":[{"_id":"AAAAAAAAAAAAAAAAA","name":"foobar","fname":"foobar","t":"c","usersCount":10,"ts":"2019-01-01T00:00:00.000Z","default":false,"lastMessage":{"_id":"AAAAAAAAAAAAAAAAA","rid":"AAAAAAAAAAAAAAAAA","msg":"Hello, World!","ts":"2019-01-01T00:00:00.000Z","u":{"_id":"AAAAAAAAAAAAAAAAA","username":"normaluser","name":"normaluser"},"unread":true,"_updatedAt":"2019-01-01T00:00:00.000Z","urls":[],"mentions":[],"channels":[]},"description":"Obviously, this JSON response is heavily censored."}],"count":25,"offset":0,"total":52,"success":true}
      ```
      
      (Obviously, this JSON response is heavily censored, but you get the gist: It was successful.)
      
      ### "Bad" request (descending order):
      
      `https://rocketchat.domain.org/api/v1/directory?query=%7B%22type%22%3A%22channels%22%2C%22text%22%3A%22%22%2C%22workspace%22%3A%22local%22%7D&sort=%7B%22lastMessage%22%3A0%7D&count=25`
      
      More easily readable GET parameters:
      
      ```
      query | {"type":"channels","text":"","workspace":"local"}
      sort | {"lastMessage":0}
      count | 25
      ```
      
      Response:
      ```
      {"success":false,"error":"Invalid sort parameter provided: \"{\"lastMessage\":0}\" [error-invalid-sort]","errorType":"error-invalid-sort","details":{"helperMethod":"parseJsonQuery"}}
      ```
      
      ## Further comments
      
      Version on the server where I noticed this: `https://rocketchat.$DOMAIN/api/info` returns `{"version":"4.8","success":true}`. According to the "Releases" page, this version appeared 5 days ago, so I believe this is up to date.
      
      ### The journey to here
      
      For some reason, the descending order uses the wrong magic number "0", and the server can't interpret that. However, this *used* to work, so I'm not very sure about this.
      
      The error message appears in the source code of the entire organization exactly once: https://github.com/RocketChat/Rocket.Chat/blob/31ae30f30ad71d9e5a1b0cad494b3471a7dd8807/apps/meteor/app/api/server/helpers/parseJsonQuery.ts#L42
      So I'll guess that this is the line of code that generated this particular message.
      
      A few lines above we see that the server only knows 1 and -1 as magic numbers for the sorting:
      https://github.com/RocketChat/Rocket.Chat/blob/31ae30f30ad71d9e5a1b0cad494b3471a7dd8807/apps/meteor/app/api/server/helpers/parseJsonQuery.ts#L35
      This matches the observed bug: The browser sends 1 (which works) and 0 (which doesn't work).
      
      Generally, it seems that the web client actually uses the strings "asc" and "desc" internally, which are hard to mix up. So I assume that it's the conversion of that is broken somehow.
      
      Exactly this seems to be the case here:
      https://github.com/RocketChat/Rocket.Chat/blob/31ae30f30ad71d9e5a1b0cad494b3471a7dd8807/apps/meteor/client/views/directory/hooks.js#L11
      
      The code around it produces exactly the kind of query seen in the network log, and can also produce the buggy parameter `sort: 0`. This either fixes bug #25695, or a different bug of the same kind.
      
      I am not sure how to add tests for this, can someone do this for me or show me where to start? I'm actually just an end-user and "accidentally" found the fix for the bug while writing the bug report ^^'
      
      EDIT: Rebased for convenience, and to re-check CI.
      Unverified
      5a37518e
  2. Jun 11, 2022
    • Weslley Campos's avatar
      Chore: Add tests for agents screens (#25637) · 0d8a2aa2
      Weslley Campos 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)
      Add some tests for omnichannel agents screen and add some data-qa in components f
      <!-- 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... -->
      Unverified
      0d8a2aa2
  3. Jun 10, 2022
    • Yash Rajpal's avatar
      Chore: Notification Preferences to TS (#25827) · ffd64572
      Yash Rajpal 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.
      -->
      
      - Notifications Preferences to TS.
      - Fix broken save action.
      
      <!-- 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... -->
      Unverified
      ffd64572
    • Júlia Jaeger Foresti's avatar
      Chore: Convert MemoizedSetting, Setting, Section (#25572) · e01c8eac
      Júlia Jaeger Foresti 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>
      Unverified
      e01c8eac
    • Guilherme Gazzo's avatar
      Regression: Fix users.create call (#25834) · e3d184bc
      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... -->
      Unverified
      e3d184bc
    • Guilherme Gazzo's avatar
      Chore: Add auto label and improve Kodiak configuration (#25829) · 5a287fff
      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... -->
      Unverified
      5a287fff
    • Henrique Guimarães Ribeiro's avatar
    • Debdut Chakraborty's avatar
      Chore: Remove compose from main repo (#23426) · e38b5bc1
      Debdut Chakraborty authored
      Closes #23584
      Closes #23416
      Closes #23402
      Closes #24636 
      Closes #24160 
      Closes #23479 
      Closes #23341
      Unverified
      e38b5bc1
  4. Jun 09, 2022
    • gabriellsh's avatar
      [FIX] `You and @yourUsername reacted with`title on reactions (#25733) · d2b214f1
      gabriellsh 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... -->
      Unverified
      d2b214f1
    • 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... -->
      Unverified
      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... -->
      Unverified
      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
      Unverified
      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... -->
      Unverified
      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... -->
      Unverified
      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
      Unverified
      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... -->
      Unverified
      11a9d235
    • Guilherme Gazzo's avatar
      Regression: fix apps path (#25809) · c27412b0
      Guilherme Gazzo authored
      Unverified
      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>
      Unverified
      993745d3
  5. 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>
      Unverified
      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 
      Unverified
      2a292e4f
    • Guilherme Gazzo's avatar
      Chore: Fix CI (#25797) · 9629831b
      Guilherme Gazzo authored
      Unverified
      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)
      Unverified
      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... -->
      Unverified
      4391ca64
    • Guilherme Gazzo's avatar
      Update .kodiak.toml · ccc869fd
      Guilherme Gazzo authored
      Unverified
      ccc869fd
    • Guilherme Gazzo's avatar
      Update .kodiak.toml · 7ea6491c
      Guilherme Gazzo authored
      Unverified
      7ea6491c
    • Guilherme Gazzo's avatar
      Chore: Testing Kodiak feature (#25794) · e675d526
      Guilherme Gazzo authored
      Unverified
      e675d526
  6. Jun 07, 2022
Loading