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

3.0 slot mappings: rasa-sdk implementation #529

Closed
3 tasks
ancalita opened this issue Aug 16, 2021 · 0 comments · Fixed by #549
Closed
3 tasks

3.0 slot mappings: rasa-sdk implementation #529

ancalita opened this issue Aug 16, 2021 · 0 comments · Fixed by #549
Assignees
Labels
area:rasa-sdk 🧑‍💻 Everything that touches our python Rasa SDK feature:3.0/slot-mappings priority:high Impacts a high number of users. Prevents using a core feature normally or secondary feature at all type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR

Comments

@ancalita
Copy link
Member

ancalita commented Aug 16, 2021

Description of Problem:
Background: This is provided in Taking Apart Forms and Slot Mappings.

Summary: The approach of filling slots in 2.0 includes an implicit mechanism (via auto-fill) and an explicit mechanism (via forms). This makes it challenging to keep information up-to-date throughout the course of a conversation, as well as requires slot mapping duplication across forms if multiple forms should fill the same slot.

Overview of the Solution:

  • The full extent of proposed changes can be found in the implementation proposal.
  • This issue is only concerned with the section requiring adaptations to the rasa-sdk repo.

Definition of Done:

  • adapt FormAction to mirror changes in rasa-oss FormAction
  • adapt FormValidationAction interface
  • changes are unit tested
@ancalita ancalita added area:rasa-sdk 🧑‍💻 Everything that touches our python Rasa SDK type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR labels Aug 16, 2021
@TyDunn TyDunn added the priority:normal Impacts some users. Prevents using a secondary feature normally. label Aug 25, 2021
@TyDunn TyDunn added priority:high Impacts a high number of users. Prevents using a core feature normally or secondary feature at all and removed priority:normal Impacts some users. Prevents using a secondary feature normally. labels Sep 10, 2021
@alwx alwx mentioned this issue Oct 4, 2021
5 tasks
@alwx alwx closed this as completed in #549 Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-sdk 🧑‍💻 Everything that touches our python Rasa SDK feature:3.0/slot-mappings priority:high Impacts a high number of users. Prevents using a core feature normally or secondary feature at all type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants