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

[Frontend] [Chat Bot] #54

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

[Frontend] [Chat Bot] #54

wants to merge 11 commits into from

Conversation

atuljayaram
Copy link
Contributor

URL: https://codesandbox.io/s/github/bitprj/mdx-deck/tree/frontend-chatbot

Issues Closed

...

Changes proposed in this pull request:

  • Current State of Chat Bot Workshop

@atuljayaram atuljayaram added the weekly-manager Contains all tasks that should be completed BY MANAGERS for the week. label Apr 16, 2020
Copy link
Contributor

@kavuong kavuong left a comment

Choose a reason for hiding this comment

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

Hey @ShayanRiyaz @Dadao-xiaoguaishou thanks for your work! The workshop is easy to follow. A couple overall things to note:

There should be explanations of every step you are taking - what exactly is being accomplished with these steps?
the context of this workshop should be shifted to fit a business theme. In other words, how can a business make a Reddit Bot to answer questions and comments about their company for PR?

As an organization, we are shifting directions towards making more practical and marketable content targeting different majors outside of STEM. A Reddit bot would be very useful for business, and we want to make all of our resources practical for different audiences.

So that should be the focus this week, as well as fixing the comments I made.
@atuljayaram please direct this process! Thanks


<GreetingLoader version="class.1.2" />
## Never miss a chance to engage with your friends or users
Copy link
Contributor

Choose a reason for hiding this comment

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

this slide can be expanded on a lot with use cases and examples


### The face of your bot.

#### Connects to your backend and provides it with business logic.
Copy link
Contributor

Choose a reason for hiding this comment

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

what does "business logic" mean?


## A chain of unfortunate consequences
### A context-aware bot can remember things, and hold conversations
Copy link
Contributor

Choose a reason for hiding this comment

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

explanation a little vague


<L.Row>
## 2. Setting up a “Welcome” Intent
Copy link
Contributor

Choose a reason for hiding this comment

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

For each of these instructions, there should be an explanation on what exactly they are setting up and not just step by step instructions

## Observing the Lifecycle.

### Overlapping the lifecycle with hooks, we see a phase that we can't observe...
## 4. Attaching our new Entity to the Intent
Copy link
Contributor

Choose a reason for hiding this comment

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

a reminder on why we are attaching an entity to an intent would be awesome!

<CodeSurfer>

​```js
'use strict';
Copy link
Contributor

Choose a reason for hiding this comment

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

this slide is not very helpful. the code should be split up into sections and each section should be briefly explained. And the whole code should be available in a GitHub Gist


<L.Row>
## Steps
Copy link
Contributor

Choose a reason for hiding this comment

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

what exactly is going on in this slide? a bit confused here.


</L.Row>
---
Copy link
Contributor

Choose a reason for hiding this comment

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

Again I think explaining what the steps are accomplishing will be helpful


### Synchronous Subscriber and scheduled teardown
- “More”
Copy link
Contributor

Choose a reason for hiding this comment

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

the quotes aren't too helpful and don't add too much here


<L.Column>
## 7. Integrations.
Copy link
Contributor

Choose a reason for hiding this comment

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

what exactly are these integrations doing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
weekly-manager Contains all tasks that should be completed BY MANAGERS for the week.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants