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

Chat: implement P2P demo #28363

Merged
merged 31 commits into from
Nov 21, 2024
Merged

Conversation

Strider2342
Copy link
Contributor

No description provided.

@Strider2342 Strider2342 requested review from EugeniyKiyashko and a team November 14, 2024 14:37
@Strider2342 Strider2342 self-assigned this Nov 14, 2024
apps/demos/Demos/Chat/Overview/jQuery/index.html Outdated Show resolved Hide resolved
apps/demos/Demos/Chat/Overview/jQuery/index.js Outdated Show resolved Hide resolved
apps/demos/Demos/Chat/Overview/jQuery/index.js Outdated Show resolved Hide resolved
apps/demos/Demos/Chat/Overview/jQuery/index.js Outdated Show resolved Hide resolved
apps/demos/menuMeta.json Outdated Show resolved Hide resolved
apps/demos/menuMeta.json Outdated Show resolved Hide resolved
apps/demos/menuMeta.json Outdated Show resolved Hide resolved
}

const userChat = $("#user-chat").dxChat({
width: 760,
Copy link
Contributor

Choose a reason for hiding this comment

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

TestCafe tests issues

  1. I got the etalons from the 'screenshot' archive. You can download them here https://github.com/DevExpress/DevExtreme/actions/runs/11931869901?pr=28363.
    Chats don't fit within the provided height of the window area.
    The input is cropped.
  2. The time for each day will be updated

Copy link
Contributor

@EugeniyKiyashko EugeniyKiyashko Nov 20, 2024

Choose a reason for hiding this comment

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

Issues are resolved in the c568242
Etalons are added in the b1e4708

@EugeniyKiyashko EugeniyKiyashko self-requested a review November 21, 2024 09:40
"Equivalents": "",
"Demos": [
{
"Title": "Overview",
Copy link
Contributor

Choose a reason for hiding this comment

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

Kindly add a "new" badge

@EugeniyKiyashko EugeniyKiyashko merged commit e9863f6 into DevExpress:24_2 Nov 21, 2024
276 of 278 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.

4 participants