Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace react-render-html library #2331

Merged
merged 4 commits into from
Oct 22, 2024

Conversation

camillof
Copy link
Contributor

@camillof camillof commented Oct 18, 2024

Description

Replaces react-render-html, which is currently no longer maintained, for a new one: https://github.com/remarkablemark/html-react-parser

More Details

  • Almost all of the show happens in Utils.renderContent
  • There were a few components calling directly to the react-render-html library, which now I've updated to call the Utils.renderContent instead
  • There shouldn't be any noticeable UI change, please help me double-checking this, as I'm new to the app :)

Corresponding Issue


Reviewing this pull request? Check out our Code Review Practices guide if you haven't already!

Comment on lines +3 to +7
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
import { Tooltip } from 'components/Tooltip';
import type { Node } from 'react';
import React from 'react';
import globalCss from 'styles/_global.scss';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No change here, just a reorder of imports

Comment on lines +3 to +5
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
import type { Node } from 'react';
import React, { useState } from 'react';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No change here, just a reorder of imports

@camillof camillof added react and removed wip labels Oct 18, 2024
@camillof camillof marked this pull request as ready for review October 18, 2024 16:41
Copy link
Member

@julianguyen julianguyen left a comment

Choose a reason for hiding this comment

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

Great work! Looks good to me!

@julianguyen julianguyen merged commit 8bb577b into main Oct 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace unmaintained react-render-html library
2 participants