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

Trying to fix two headless instances being in build #3358

Merged
merged 7 commits into from
Apr 2, 2024

Conversation

jcbrand
Copy link
Member

@jcbrand jcbrand commented Mar 28, 2024

No description provided.

@jcbrand jcbrand force-pushed the jcbrand/fix-two-instances-headless branch 3 times, most recently from 396cafa to f7dafb5 Compare April 1, 2024 21:10
jcbrand added 6 commits April 2, 2024 10:36
Refactored so that all imports from `@converse/headless` are from the
index and not deeper files.

Updates #3348
Remove the `browser` entry in the headless package.json, otherwise the
headless build (in src/headless/dist) is used when building converse.js,
which tries to load a different emojis.json
@jcbrand jcbrand force-pushed the jcbrand/fix-two-instances-headless branch from 4485956 to 3e96a46 Compare April 2, 2024 08:37
@jcbrand jcbrand force-pushed the jcbrand/fix-two-instances-headless branch from f47a7dc to 41995f2 Compare April 2, 2024 08:40
@jcbrand jcbrand merged commit 7790830 into master Apr 2, 2024
4 checks passed
@jcbrand jcbrand deleted the jcbrand/fix-two-instances-headless branch January 12, 2025 20:20
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.

1 participant