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 907 commits into
base: main
Choose a base branch
from
Open

EmbeddedChat enhancements #219

wants to merge 907 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.

Spiral-Memory and others added 23 commits June 24, 2024 13:35
…essing

feat: UIKit improvement and action processing
modular code

handled auth flow in EC API
…ing_screens

fix/feat: Improve auto login + loading screens
* Fixed unwanted close call in ChatBody.js

* Separated the useEffect which calls getMessagesAndRoles

* Formatted using prettier
* Fixed: The read-only is also set for admins which should not be the case.

* fix: enable messaging for admins in read-only rooms based on workspace permissions

* Formatted using prettier
* Fix:Fix: Restrict pin icon visibility based on user permissions

* Restrict pin icon visibility
* fixed recursive quoting issue

* fixed multiple quoting issue

* Removed unwanted comment

* added checks which causing error in deployment

* Resolved the commits

* Fix the issue of not able to send the quoted thread messages
* Added go to Message Feature

* fix linting issues

* Implement jump to message functionality

* Fix linting

* Append msg to URL

* Remove appending in URL

* Fix popup mode issue
…es. (#669)

* fetch starred message api called

* starred messages fetching from api

* ran prettier

* fixed linting errors
* Fixed UI issues for Sidebar

* Fixed sidebar height issue

* Removed unnecessary spacing

* Fixed reply in thread issue

* Adjusted dynamic header visibility

* Revert zIndex changes

* Linting fix

* Run prettier

* Remove zIndex

* Adjusted zIndex for ViewComponent

* Fix sidebar width

* Remove global css for sidebar

---------

Co-authored-by: Zishan Ahmad <[email protected]>
* fix mardown

* fix conflicts

* fix formatting issues

* fixed lint issues

* changes

* prettier

* fix quote

* fixed

* removed markdown from passed prop

---------

Co-authored-by: Zishan Ahmad <[email protected]>
* fix(chat): make mentions interactive with user detail modal

* Remove unnecessary space

* Ran format and lint
)

* Added a feature to copy messages

* Added copy link feature

* fixed inconsistency in layout_editor

* Removed changes in layout_editor
)

* Added a validation check for pr approval in the deploy-pr workflow

* Added success() call in downloading artifact and deploy to github-pages steps

* Used Github API to get the pull request number in deploy-pr workflow

* fetching pr number using github user reference
* Implemented search member and display member info feature

* Fix UserInfo panel width

* Added padding to sidebar contents

---------

Co-authored-by: Zishan Ahmad <[email protected]>
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.

8 participants