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

chore(core): Remove unused code from experimental 1.0 version #2812

Merged
merged 8 commits into from
May 6, 2024

Conversation

vanbasten17
Copy link
Contributor

@vanbasten17 vanbasten17 commented Apr 23, 2024

Description

@botonic/core
Add functions like isWhatsapp, pass the session and check the provider. Better if they are in @botonic/core as we can use them in several packages and also in the bot.

@botonic/react
Use isWhatsapp and isFacebook from core

@botonic/dx
Add the jsdom package, in the latest version of jest it does not come by default, you need to install a separate package.

Use the tsconfig in the transform attribute of jest.config together with ts-jest. The latest version of jest cannot use this configuration in the globals attribute.

Context

Approach taken / Explain the design

To document / Usage example

Testing

The pull request...

  • has unit tests
  • has integration tests
  • doesn't need tests because... [provide a description]

Copy link

linear bot commented Apr 23, 2024

@vanbasten17 vanbasten17 changed the title chore(core): fix some types chore(core): Remove unused code from experimental 1.0 version Apr 23, 2024
@vanbasten17 vanbasten17 marked this pull request as ready for review May 2, 2024 07:25
@vanbasten17 vanbasten17 force-pushed the feature/blt-691-task-review-botonic-core branch from f475f62 to 7ca58d5 Compare May 2, 2024 07:31
Copy link

github-actions bot commented May 2, 2024

Test Results

30 tests  ±0   30 ✅ ±0   5m 0s ⏱️ -12s
 6 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 7ca58d5. ± Comparison against base commit 479b890.

♻️ This comment has been updated with latest results.

@vanbasten17 vanbasten17 merged commit e507007 into master-lts May 6, 2024
4 of 5 checks passed
@vanbasten17 vanbasten17 deleted the feature/blt-691-task-review-botonic-core branch May 6, 2024 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants