Skip to content

Parse Topic Comments#3

Open
ekajl wants to merge 2 commits into
RyverApp:masterfrom
ekajl:parse_post_comments
Open

Parse Topic Comments#3
ekajl wants to merge 2 commits into
RyverApp:masterfrom
ekajl:parse_post_comments

Conversation

@ekajl
Copy link
Copy Markdown

@ekajl ekajl commented Feb 9, 2018

Added conditional arguments to recognize the API that is seen when a comment is made in a topic. Under the handleEvent function, the final else if statement that is matched on the '/api/activityfeed/postComments/changed' string begins the process of filtering through a comment, to determine if the bot should respond in the topic. This is then sent to the new handleCommentChange function which declares two variables, the comment's ID and the post's ID. From there, it is passed to commentChangeParse which executes a GET request on the Ryver API for the comment that was made. The response JSON is then dealt with, sending the topic's ID, comment text, and comment ID to the middleware for regex evaluation.

Copy link
Copy Markdown
Author

@ekajl ekajl left a comment

Choose a reason for hiding this comment

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

Removed some other functions I am working on separately for recognizing @mentions within a topic's comment

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.

1 participant