Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Conversation

nawarajshah
Copy link
Contributor

@nawarajshah nawarajshah commented Jan 28, 2023

Fixing the issue of element-hq/element-web#23837

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

Signed-off-by: Nawaraj Shah [email protected]


This PR currently has none of the required changelog labels.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

@nawarajshah nawarajshah requested a review from a team as a code owner January 28, 2023 00:13
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Jan 28, 2023
@nawarajshah
Copy link
Contributor Author

before
Screenshot (29)

after
Screenshot (26)

Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

This code is used throughout the whole product and not only in the context of chat exports.

You will also need to write tests for this PR to be merged as per then contributing guidelines https://github.com/vector-im/element-web/blob/develop/CONTRIBUTING.md

@germain-gg germain-gg removed the request for review from kerryarchibald January 30, 2023 10:01
make changes on test cases by adding angel bracket around the sender
@richvdh
Copy link
Member

richvdh commented May 3, 2023

Thanks for your contribution. I'm going to close this for now as the changes requested haven't been made. If you're interested in continuing to work on this please let us know and we can reopen the pull request. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants