fix(actions): removed tooltip with doc of deprecated api #326
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DEV-2083 [Bug] Newly added actions do not appear in the list
It can be a documentation problem or a code problem, not sure. However, a user already pointed to this problem in the following post https://forum.botpress.com/t/new-actions-dont-appear/2512 … and it was 2019…. Steps to reproduce 1- Create a new js script and add it to the /packages/bp/dist/data/bots//actions folder in my test I added the CarlesBotAPI2.js to the folder, and as expected it shows in the actions folder 2- Now try to add this new action to a node by selecting a node and adding a new action Actual Results The new action does not appear in the list Expected result The new action is listed in the drop down. As a user, the behaviour is not what I expect, however, there may be reasons behind that justify this behaviour. However, hovering the cursor I can read the following message I have googled 'bp.registerActions()' and the first hit I find is the forum question that I wrote above, so the message is not really helpful and I would suggest to change it to a url or something that can really help the user. |
samuelmasse
approved these changes
Mar 29, 2022
samuelmasse
added a commit
that referenced
this pull request
Apr 22, 2022
* feature(studio-ui): deploy chatbot button (#299) * merge next * first version deploy cloud btn * polish button, better exp * env var for link * i18n * ui bug changes * spelling errors Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * Spelling Errors Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * Fr Spelling Mistake Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * Fr Spelling Mistake Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * todo move hardcoded value * FR Change wording * refactor functions * refactor, more error codes * eslint * ignore libraries node modules Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> Co-authored-by: allardy <yann.allard1@gmail.com> * fix(studio): remove skill unusable on cloud (#316) * feat(studio): allow disabling file listeners for read-only deployments (#242) * fix(code-editor): minor ux fixes (#313) * chore(studio-be): add route to invalidate cms cache for bot (#318) * fix(code-editor): slight adjustment (#319) * chore(server): v0.0.49 (#320) * chore(inspector-kit): migrate superinput package to inspector-kit (#309) * chore(inspector-kit): change file structure * chore(inspector-kit): switch to scss modules * feat(inspector-kit): finish setup; stub provider * chore(inspector-kit): add storybook (#314) * chore(inspector-kit): change file structure * chore(inspector-kit): switch to scss modules * chore(inspector-kit): add storybook * fix(ts): wrong types for react-router-dom * Implement text field as standalone (#315) * chore(inspector-kit): change file structure * chore(inspector-kit): add storybook * chore(inspector-kit): add input; component and stories * chore(inspector-kit): inherit from bp3 input; support leftIcon, rightElement * Bootstrap inspector-kit tests (#321) * feat(inspector-kit): add RTL * chore(superinput): add tests * chore(inspector-kit): update readme * chore(migration): migration for srv_channels (#324) * chore(migration): migration for srv_channels * v0.0.50 * bump * down migrate * update mig * fix typo * fix(hooks): send events to core on bot mount and unmount (#322) * update botpress.d.ts * fix(actions): removed tooltip with doc of deprecated api (#326) * feat(studio): add download modelweights step before uploading chatbot to cloud (#325) * feat(studio): add download modelweights step before uploading chatbot to cloud * removed typo * pr comment * fix(actions): fix number of conversations action (#328) * chore(core): use proper env var for controller endpoint (#329) * fix(dev): local cdm setup (#330) * fix(dev): local cdm setup * fix * chore(dev): cloud local * fix(debugger): do not escape user input in debugger logs (#327) * fix(scope): call oauth with nlu scope (#334) * chore(server): release v0.0.51 (#335) * fix(code-editor): allow creating hooks with hook type as name (#333) * feat(deploy): deploy to cloud btn should show failure toaster if runtime not ready (#336) * feat(deploy): deploy to cloud btn should show failure toaster if runtime not ready * chore(translations): add translations fr-en and point to those * chore(es): add es translation * feat(testing): added testing module to studio (#331) * add missing translation * fix(ui): toggle explorer (#338) Co-authored-by: Laurent Leclerc-Poulin <laurentleclercpoulin@gmail.com> * fix(cloud): cloud router error handling (#339) * fix(cloud-service): error handling * fix(ui): better handle push to cloud error codes * Update packages/studio-be/src/studio/cloud/cloud-router.ts Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * Update packages/studio-be/src/studio/cloud/cloud-router.ts Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * Update packages/studio-be/src/studio/cloud/cloud-router.ts Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * custom error handling * remove libraries feom sidebar Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> * chore: remove libraries module (#342) * wtf * yarn * fix * fixes Co-authored-by: Nick B <boutboulnicholas@gmail.com> Co-authored-by: Laurent Leclerc Poulin <laurentlp@users.noreply.github.com> Co-authored-by: allardy <yann.allard1@gmail.com> Co-authored-by: Yann Allard <42552874+allardy@users.noreply.github.com> Co-authored-by: David Vitor Antonio <david.vitora@gmail.com> Co-authored-by: Laurent Leclerc-Poulin <laurentleclercpoulin@gmail.com> Co-authored-by: Rob Gordon <rchristopher.gordon@gmail.com> Co-authored-by: Thomas Caissie <30733005+tomkcey@users.noreply.github.com> Co-authored-by: Francois-Xavier P. Darveau <955524+EFF@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR removes the tooltip displayed when adding a new action that would recommend using an API that has been deprecated for a long time.
Before
After