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-cli: no read/write bot.config.json #2895

Merged

Conversation

Iru89
Copy link
Contributor

@Iru89 Iru89 commented Aug 27, 2024

Description

  • Not read or write the bot.config.json file, just get the bot config values to send them in the post when deploying.
  • Use the ora spinner to give feedback to the user: Getting bot config...

Copy link

Test Results

30 tests  ±0   30 ✅ ±0   4m 53s ⏱️ -24s
 6 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 1a3f5fe. ± Comparison against base commit e0e6ce4.

const packages = await this.readPackageJson()
static async get(appDirectory: string): Promise<BotConfigJSON> {
const spinner = ora({
text: 'Getting bot config...',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would remove this message, I think it is not needed for the user to know this is happening in the background

@Iru89 Iru89 merged commit 5961ceb into botonic-cli/axios-with-interceptors Aug 27, 2024
4 of 5 checks passed
@Iru89 Iru89 deleted the botonic-cli/no-write-bot-config-json branch August 27, 2024 11:40
Iru89 added a commit that referenced this pull request Oct 1, 2024
## Description

When deploying a bot the cli will get bot config from the @Botonic
packages installed, npm and node version .
This config is send to backend in deploy endpoint. 

## Context

This config will be used to read bot information from the flow builder
frontend.

## Approach taken / Explain the design

PRs merged in this PR:

- #2895
- #2894
- #2892
- #2885
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