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

Added a table explaining all the different fields that are used while creating a new conversation logic. #28

Conversation

chinmoy12c
Copy link
Member

@chinmoy12c chinmoy12c commented Apr 18, 2023

| Key | Type | Required | Description |
| --- | --- | --- | --- |
| data | object | Yes | The root object of the data. |
| data.id | null | No | The id of the data. |

Choose a reason for hiding this comment

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

What should be the format of the ID?

Copy link
Member Author

Choose a reason for hiding this comment

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

Can you help me out with this? The example request you shared had null in this field.

Copy link
Member Author

@chinmoy12c chinmoy12c May 3, 2023

Choose a reason for hiding this comment

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

This field is not being used in the request and the id corresponds to the id of the bot in the database which is autogenerated. If this field is set and the id corresponds to an existing bot, an error is thrown.

@chinmoy12c chinmoy12c force-pushed the docs/add_conversation_logic_request_table branch 2 times, most recently from 0f2575c to f0063cf Compare May 3, 2023 04:31
@chinmoy12c
Copy link
Member Author

@karntrehan I've updated the PR.

@chinmoy12c chinmoy12c force-pushed the docs/add_conversation_logic_request_table branch from f0063cf to d12bf9c Compare May 3, 2023 04:55
@chinmoy12c chinmoy12c merged commit e736da9 into samagra-comms:main Jun 19, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants