Skip to content

Comments

Added support for the <br> tag and support for nested assets in the t…#14

Merged
sunil-lakshman merged 8 commits intonextfrom
Enh/CS-39183-nested-assets-in-image
Jun 26, 2023
Merged

Added support for the <br> tag and support for nested assets in the t…#14
sunil-lakshman merged 8 commits intonextfrom
Enh/CS-39183-nested-assets-in-image

Conversation

@sunil-lakshman
Copy link
Contributor

No description provided.

@sunil-lakshman sunil-lakshman requested a review from a team as a code owner June 19, 2023 09:45
@github-actions
Copy link

We regret to inform you that you are currently not able to merge your changes into the master branch due to restrictions applied by our SRE team. To proceed with merging your changes, we kindly request that you create a pull request from the next branch. Our team will then review the changes and work with you to ensure a successful merge into the master branch.

@sunil-lakshman sunil-lakshman changed the base branch from master to next June 19, 2023 09:47
@ishaileshmishra
Copy link
Member

  • Test cases missing, Please add test cases for the break line.
  • Code missing for image
  • break code implementation missing

Copy link
Member

@ishaileshmishra ishaileshmishra left a comment

Choose a reason for hiding this comment

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

Please refer the comment

@sunil-lakshman
Copy link
Contributor Author

  • Test cases missing, Please add test cases for the break line.
  • Code missing for image
  • break code implementation missing

Code implementation done and added test cases.

Copy link
Member

@ishaileshmishra ishaileshmishra left a comment

Choose a reason for hiding this comment

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

Added on minor change

@ishaileshmishra
Copy link
Member

ishaileshmishra commented Jun 21, 2023

Reviewed, found code implementation missing. Please complete the code implementation @sunil-lakshman

@sunil-lakshman
Copy link
Contributor Author

Reviewed, found code implementation missing. Please complete the code implementation @sunil-lakshman

Done

Copy link
Member

@ishaileshmishra ishaileshmishra left a comment

Choose a reason for hiding this comment

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

Approved.

Please share the pull request (PR) details with Gautam for the CAB and also submit a request to the SRE team for code review.

Copy link
Member

@ishaileshmishra ishaileshmishra left a comment

Choose a reason for hiding this comment

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

approved

@sunil-lakshman sunil-lakshman merged commit e726150 into next Jun 26, 2023
sunil-lakshman added a commit that referenced this pull request Jun 28, 2023
#14) (#15)

* Added support for the <br> tag and support for nested assets in the the image

* Added version and change log

* Added Testcases

* Resolved dependency gem issues in latest ruby versions

* Added test case for break tag

* Changed tag to full name

* Changed method name from 'text_to_htms' to 'text_to_html'

* Changed version from 1.1.4 to 1.2.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants