Skip to content

make emojis.json a ts file so no bad path in type defs#304

Open
revilossor wants to merge 2 commits intoealush:masterfrom
revilossor:master
Open

make emojis.json a ts file so no bad path in type defs#304
revilossor wants to merge 2 commits intoealush:masterfrom
revilossor:master

Conversation

@revilossor
Copy link
Copy Markdown

@revilossor revilossor commented Nov 30, 2022

writes 'emojis.ts' instead of 'emojis.json' so that there is no bad path to './emojis.json' in dist/ ( the json is only in src/ )

closes [https://github.com//issues/294] i think - i was seeing the same error in a library project with typescript ( i have a typescript build as a git hook - the rollup build worked fine, guess cos rollup doesnt need to resolve emojis.json in the ts defs )

@arjita-mitra
Copy link
Copy Markdown

arjita-mitra commented May 10, 2023

@ealush Can we merge this? I am having same issues, my build pipeline keeps failing.

Run set -o pipefail
Error: node_modules/emoji-picker-react/dist/data/emojis.d.ts(1,20): error TS2307: Cannot find module './emojis.json' or its corresponding type declarations.
Error: Process completed with exit code 2.

the json file doesn't exist under dist directory, its only present under src hence the problem. this will prevent me from using the library and our release date is close. Please help me resolve this issue. Note, I can't add skipLibcheck in this project.

@ealush ealush force-pushed the master branch 2 times, most recently from 8673ce9 to d8ad1d4 Compare November 4, 2023 20:16
@ealush ealush force-pushed the master branch 8 times, most recently from e1e9bc5 to 76f2b14 Compare October 19, 2025 22:14
@ealush ealush force-pushed the master branch 5 times, most recently from 6f0292c to d2af775 Compare November 22, 2025 13:03
@ealush ealush force-pushed the master branch 2 times, most recently from 467cac6 to d55c0c3 Compare January 22, 2026 21:30
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.

2 participants