Skip to content
This repository has been archived by the owner on Jan 14, 2020. It is now read-only.

Review listener #67

Merged
merged 8 commits into from
May 21, 2018
Merged

Review listener #67

merged 8 commits into from
May 21, 2018

Conversation

Gzing
Copy link

@Gzing Gzing commented May 21, 2018

First pull request? Read our guide to contributing

Checklist:

  • Code contains relevant tests for the problem you are solving
  • Code has been formatted with autopep8 --in-place --recursive --a --a .
  • Ensure all new and existing tests pass
  • Update any relevant READMEs and docs
  • Submit to the develop branch instead of master

Description:

Please explain the changes you made here:

  • Adds event listner, db service and API endpoint for Reviews

@crazybuster crazybuster self-requested a review May 21, 2018 21:16
@Gzing Gzing changed the base branch from master to develop May 21, 2018 21:31
crazybuster
crazybuster approved these changes May 21, 2018
View reviewed changes
Copy link
Contributor

@crazybuster crazybuster left a comment

Choose a reason for hiding this comment

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

Great, we really need to get our contracts synced up with what's current on origin-js would be nice if there's some versioning/tagging systems to sync the two.

@crazybuster crazybuster merged commit 0baaa25 into develop May 21, 2018
@crazybuster crazybuster deleted the review_listener branch May 21, 2018 21:43
@micahalcorn micahalcorn mentioned this pull request May 22, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@crazybuster crazybuster crazybuster approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Gzing @crazybuster