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

botonic-react: remove getConfig function #BLT-1036 #2902

Merged
merged 2 commits into from
Oct 1, 2024

Conversation

Iru89
Copy link
Contributor

@Iru89 Iru89 commented Sep 13, 2024

Description

Remove the getConfig function that is run to get the response from the botonic-config lambda.

Context

In the new version of cli we will create the bot configuration at the moment of deploying, it will be saved in the database and will be updated in each deploy. We will not only have the plugins configuration.

@Iru89 Iru89 changed the title botonic-react: remove getConfig function botonic-react: remove getConfig function #BLT-1036 Sep 13, 2024
@Iru89 Iru89 force-pushed the BLT-1036-remove-botonic-get-config branch from be12e94 to 4627692 Compare October 1, 2024 15:35
@pull-request-size pull-request-size bot added size/S and removed size/XS labels Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Test Results

30 tests  ±0   26 ✅ ±0   33s ⏱️ -3s
 6 suites ±0    0 💤 ±0 
 1 files   ±0    4 ❌ ±0 

For more details on these failures, see this check.

Results for commit 4627692. ± Comparison against base commit e08d09f.

♻️ This comment has been updated with latest results.

@Iru89 Iru89 merged commit 39b3f98 into master-lts Oct 1, 2024
16 of 18 checks passed
@Iru89 Iru89 deleted the BLT-1036-remove-botonic-get-config branch October 1, 2024 15:38
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.

3 participants