Skip to content

Conversation

@arnautov-anton
Copy link
Contributor

@arnautov-anton arnautov-anton commented Aug 24, 2022

🎯 Goal

Vite development mode is broken for the latest version of the SDK (when installed from the NPM) when using "custom" object (with default property) dynamic imports. I've re-exported the emoji-mart modules based on this comment of one somewhat unrelated issue on GH and that seems to fix the issue. Used /dist-es instead of /dist which comes with the benefit of sligtly smaller bundle size.

🛠 Implementation details

// this approach (as suggested by the aforementioned comment) increases bundle size
// so I chose different approach 
export { NimblePicker as default } from 'emoji-mart';

@arnautov-anton arnautov-anton marked this pull request as draft August 24, 2022 15:24
petyosi
petyosi previously approved these changes Aug 24, 2022
@codecov
Copy link

codecov bot commented Aug 24, 2022

Codecov Report

Merging #1724 (a31e625) into develop (515298d) will not change coverage.
The diff coverage is 100.00%.

❗ Current head a31e625 differs from pull request most recent head 6f094af. Consider uploading reports for the commit 6f094af to get more accurate results

@@           Coverage Diff            @@
##           develop    #1724   +/-   ##
========================================
  Coverage    79.63%   79.63%           
========================================
  Files          229      231    +2     
  Lines         5795     5795           
  Branches      1643     1643           
========================================
  Hits          4615     4615           
  Misses        1039     1039           
  Partials       141      141           
Impacted Files Coverage Δ
src/context/DefaultEmoji.ts 100.00% <100.00%> (ø)
src/context/DefaultEmojiPicker.ts 100.00% <100.00%> (ø)
src/context/EmojiContext.tsx 70.37% <100.00%> (-3.83%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@arnautov-anton arnautov-anton requested a review from petyosi August 24, 2022 16:45
@arnautov-anton arnautov-anton marked this pull request as ready for review August 25, 2022 14:44
@arnautov-anton arnautov-anton merged commit c90cf4b into develop Aug 26, 2022
@arnautov-anton arnautov-anton deleted the fix/vite-dynamic-import branch August 26, 2022 09:30
@MartinCupela MartinCupela mentioned this pull request Aug 29, 2022
github-actions bot pushed a commit that referenced this pull request Aug 29, 2022
# [9.5.0](v9.4.1...v9.5.0) (2022-08-29)

### Bug Fixes

* include mdast-util-find-and-replace into our CJS bundle ([#1702](#1702)) ([61c4eec](61c4eec)), closes [#1698](#1698)
* prevent double submissions in korean ([#1720](#1720)) ([5d781d8](5d781d8))
* replace FileReader with URL.createObjectURL ([#1701](#1701)) ([c8a490e](c8a490e))
* **Vite:** add emoji-mart (emoji, picker) re-export ([#1724](#1724)) ([c90cf4b](c90cf4b))

### Features

* increase and support overriding jump to message limit ([#1718](#1718)) ([8c720f4](8c720f4))
@petyosi
Copy link
Contributor

petyosi commented Aug 29, 2022

🎉 This PR is included in version 9.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Sep 6, 2022
# [10.0.0-theming-v2.3](v10.0.0-theming-v2.2...v10.0.0-theming-v2.3) (2022-09-06)

### Bug Fixes

* include mdast-util-find-and-replace into our CJS bundle ([#1702](#1702)) ([#1703](#1703)) ([8010889](8010889)), closes [#1698](#1698)
* prevent double submissions in korean ([#1720](#1720)) ([5d781d8](5d781d8))
* ThemingV2 beta adjustments ([#1728](#1728)) ([785ee11](785ee11))
* **ThemingV2:** MessageInputFlat missing lodash/zipObject ([#1721](#1721)) ([dd8a457](dd8a457))
* **Vite:** add emoji-mart (emoji, picker) re-export ([#1724](#1724)) ([c90cf4b](c90cf4b))

### Features

* increase and support overriding jump to message limit ([#1718](#1718)) ([8c720f4](8c720f4))
* **ThemingV2:** PopperTooltip component ([#1714](#1714)) ([9b6301e](9b6301e))
@petyosi
Copy link
Contributor

petyosi commented Sep 6, 2022

🎉 This PR is included in version 10.0.0-theming-v2.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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