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

EmbeddedChat enhancements #219

Open
wants to merge 890 commits into
base: main
Choose a base branch
from
Open

EmbeddedChat enhancements #219

wants to merge 890 commits into from

Conversation

sidmohanty11
Copy link
Collaborator

No description provided.

@sidmohanty11 sidmohanty11 changed the title [WIP][GSoC-2023] EmbeddedChat enhancement EmbeddedChat enhancement GSoC 2023 Sep 25, 2023
outputElement.innerHTML = `<code>\nError: ${escapedMessage}\nStack: ${escapedStack}</co>`;
}
} else {
outputElement.innerHTML = "\n" + JSON.stringify(result, null, 2);

Check warning

Code scanning / CodeQL

Exception text reinterpreted as HTML Medium

Exception text
is reinterpreted as HTML without escaping meta-characters.
@sidmohanty11 sidmohanty11 marked this pull request as ready for review February 17, 2024 17:39
@sidmohanty11
Copy link
Collaborator Author

@abhinavkrin lets get this in! We can publish a stable release and after that go with further enhancements, what do you think?

@abhinavkrin
Copy link
Member

Yes, We should make embeddedchat stable. I think we have enough features/enhancements for now. There are lots of things that need to be fixed. We will fix them and release them as the next major version.

#637)

* Combine build steps for react, ui-elements, and layout_editor packages

* Small change in build-lint
* Added Feature to deploy previews of pull requests

* Removed environment variables from build-lint workflow

* Modified the changes mentioned in the code-review
…644)

* added memoization to stop rerenders

* added memoization for allmessage and shouldrender

* fixed linting errors and imports
* fix

* formatting fix

* fix

* Fiexed lint issues
* prevent quote message overflow

* Removed sidebar open checking and added styling to Quote Message Box
…657)

* trim whitespaces in username/email fields to prevent login issues

* Remove extra .trim()
…ection (#663)

* Fix: Display emoji in visual format and enable send icon on emoji selection

* Revised code
* set messages and avatar url to null on logout

* Fixed linting issues

* Remove unnecessary import
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.

9 participants