Skip to content

[Chat] Remove file card group message from stable API#3940

Merged
Leah-Xia-Microsoft merged 5 commits intomainfrom
leahxia/RemoveFileCardGroupMessageFromStableAPI
Dec 21, 2023
Merged

[Chat] Remove file card group message from stable API#3940
Leah-Xia-Microsoft merged 5 commits intomainfrom
leahxia/RemoveFileCardGroupMessageFromStableAPI

Conversation

@Leah-Xia-Microsoft
Copy link
Copy Markdown
Member

What

Remove FileCardGroupMessage From Stable API

Why

How Tested

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

Chat bundle size is increased❗.

  • Current size: 1527206
  • Base size: 1527205
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

CallWithChat bundle size is increased❗.

  • Current size: 6424573
  • Base size: 6424572
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

Calling bundle size is increased❗.

  • Current size: 5613828
  • Base size: 5613827
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 21800 / 34332
63.49%
21800 / 34332
63.49%
595 / 1028
57.87%
1733 / 2740
63.24%
Current 21792 / 34323
63.49%
21792 / 34323
63.49%
594 / 1027
57.83%
1704 / 2722
62.6%
Diff -8 / -9
0%
-8 / -9
0%
-1 / -1
-0.04%
-29 / -18
-0.64%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 43699 / 70145
62.29%
43699 / 70145
62.29%
906 / 1970
45.98%
2585 / 4191
61.67%
Current 43728 / 70148
62.33%
43728 / 70148
62.33%
906 / 1970
45.98%
2574 / 4179
61.59%
Diff 29 / 3
0.04%
29 / 3
0.04%
0 / 0
0%
-11 / -12
-0.08%

Comment thread packages/react-components/src/components/FileDownloadCards.tsx
Co-authored-by: JoshuaLai <9044372+JoshuaLai@users.noreply.github.com>
Signed-off-by: Leah Xia <107075081+Leah-Xia-Microsoft@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@Leah-Xia-Microsoft Leah-Xia-Microsoft enabled auto-merge (squash) December 21, 2023 22:39
@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@Leah-Xia-Microsoft Leah-Xia-Microsoft merged commit 74a97ba into main Dec 21, 2023
@Leah-Xia-Microsoft Leah-Xia-Microsoft deleted the leahxia/RemoveFileCardGroupMessageFromStableAPI branch December 21, 2023 22:58
Leah-Xia-Microsoft added a commit that referenced this pull request Jan 16, 2024
* Revert "[Chat] Remove file card group message from stable API (#3940)"

This reverts commit 74a97ba.

* Revert "[Chat] Rename ImageGallery related Chat Theme (#3936)"

This reverts commit 98b9f7d.

* Revert Inline Image GA changes

* Fix lint issues

* Change files

* Duplicate change files for beta release

* Separate file sharing with inline image

* Remove test CC

* Update FileDownloadCards.test.tsx

* Update ChatScreen.tsx
palatter added a commit that referenced this pull request Jan 18, 2024
* [Calling sounds] Stabilize Calling sounds feature (#3969)

* stabilize sounds

* Change files

* Duplicate change files for beta release

* build beta API

* fix breaking change

* build API files

* update stable API

---------

Signed-off-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>

* Fix `noInvalidAny` chat composites (#3987)

* [DTMF Dialler] Enable DTMF mode and dialing out mode in dialpad (#3970)

* remove code forcing end of number to be edit with key

* Change files

* Duplicate change files for beta release

* add editing mode for dialing and dtmf default

* disable input on click if there is no editing

* remove ref

* build API

* fix test for new API

* [InlineImage][Test] Resolve resource not found error for inline image placeholder UITest  (#3979)

* Resolve resource not found error for inline image placeholder UITest

* Update UI Testing doc

Signed-off-by: Patrick Latter <73612854+palatter@users.noreply.github.com>

* Resolve issue with inline image placeholder test

* Change files

* Duplicate change files for beta release

---------

Signed-off-by: Patrick Latter <73612854+palatter@users.noreply.github.com>

* Fix up typing on chat components (#3988)

* Replace html-to-react with html-react-parser (#3995)

* Enable noImplicitAny in chat-stateful-client (#3998)

* Enable noImplicitAny in the chat sample (#4000)

* Update component examples for noImplicitAny (#4001)

* Make Calling Sample textfield validate the Teams user id format (#3967)

* Prevent dialpad unecessarily triggering onChange event (#3963)

* [DTMF Dialer] Update Dialpad styles per designs (#3999)

* Update Dialpad styles per designs

* Change files

* Duplicate change files for beta release

* fix alignment

* Update packages/react-composites CallWithChatComposite browser test snapshots

* Update packages/react-composites CallComposite browser test snapshots

* sizing

* Update packages/react-composites CallWithChatComposite browser test snapshots

* Update packages/react-composites CallComposite browser test snapshots

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* [Calling] Send reaction button component (#3997)

* Add Spotlight feature to stateful call client (#3993)

* Add Spotlight feature to stateful call client

* Change files

* Duplicate change files for beta release

* update api

* mark spot light feature as in progress

* added code docs

* Add spotlight feature selectors and handlers (#4004)

* Add Spotlight feature to stateful call client

* Change files

* Duplicate change files for beta release

* update api

* mark spot light feature as in progress

* update api

* Add Spotlight selectors and handlers

* Change files

* Duplicate change files for beta release

* remove ununsed import

* remove duplicate handlers already in common calling handlers

* memoize spotlightedParticipantIds

* remove 'on' prefix from spotlight adapter functions

* fix conditional compilation

* added code docs

* small edit

* Change hidesurvey to disablesurvey based on ARB  (#4006)

* hide survey

* Change files

* Duplicate change files for beta release

---------

Co-authored-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>

* [Calling Deps] Update to new Calling beta (#4008)

* update to new Calling beta

* Change files

* Duplicate change files for beta release

* cc new type

* fix hermetic

* fix test mock

* Update packages/react-composites ChatComposite browser test snapshots

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Add callback when survey is dismissed for selfhost  (#4009)

* add callback for survey dismissed to selfhost

* Change files

* Duplicate change files for beta release

* build files

* [Calling] Send reaction component calling-component-binding and composite changes (#4003)

* Add license field for package.json files (#4018)

* [Chat] Remove inline image from stable (#4010)

* Revert "[Chat] Remove file card group message from stable API (#3940)"

This reverts commit 74a97ba.

* Revert "[Chat] Rename ImageGallery related Chat Theme (#3936)"

This reverts commit 98b9f7d.

* Revert Inline Image GA changes

* Fix lint issues

* Change files

* Duplicate change files for beta release

* Separate file sharing with inline image

* Remove test CC

* Update FileDownloadCards.test.tsx

* Update ChatScreen.tsx

* Merge strings back into main (#4025)

* TDBuild - updating localized resource files.

* Improve CI workflow (#4016)

* Lock Files

* Change files

* Duplicate change files for beta release

* Update lock files

* Skip compare jest coverage and compare bundle size CI steps for PRs to release (#4026)

* Skip jest test coverage and compare bundle stats steps when base branch is a release branch

* Change files

* Duplicate change files for beta release

* [OCE] Remove beta banner for features that are stable (#4031)

* remove sounds beta banner

* Change files

* Duplicate change files for beta release

* [Calling] Applying capability for Reaction feature (#4013)

* Fix calling sample joining room with room id query arg (#4032)

* Fix calling sample to be able to join room with roomId query arg

* Change files

* Duplicate change files for beta release

* Update packages/react-composites ChatComposite browser test snapshots

* Update packages/react-composites ChatComposite browser test snapshots

---------

Signed-off-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>
Signed-off-by: Patrick Latter <73612854+palatter@users.noreply.github.com>
Co-authored-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>
Co-authored-by: Emlyn Bolton <3941071+emlynmac@users.noreply.github.com>
Co-authored-by: James Burnside <2684369+JamesBurnside@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Mohtasim <99507832+Mohtasim@users.noreply.github.com>
Co-authored-by: mgamis-msft <79475487+mgamis-msft@users.noreply.github.com>
Co-authored-by: carocao-msft <96077406+carocao-msft@users.noreply.github.com>
Co-authored-by: vhuseinova-msft <98852890+vhuseinova-msft@users.noreply.github.com>
Co-authored-by: Leah Xia <107075081+Leah-Xia-Microsoft@users.noreply.github.com>
Co-authored-by: edwardlee-msft <edwardlee@microsoft.com>
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.

3 participants