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

Add EasyGift to detector #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nathanho506
Copy link

Why?

Why are you doing this? What problem is it solving? Why is is good for our merchants? etc. If you're doing something from Ora PM then you can just copy the "why?" part here.

What?

What does this PR actually add. What code did you change? Did you introduce any new patterns? Etc.

Demo / Screenshots

Using recordit.co and/or monosnap to give screenshots and gif demos of your code. You can embed them in here like this: ![Brief description of image](https://www.fera.ai/wp-content/uploads/2017/12/cdyUNSv9bq.gif)
Brief description of image

How was it tested?

What tests did you run? What scenarios did you test. What tests do you still need to run?

How can you test it.

How can the reviewer test this PR? Are there any special steps required? For example:

  • bundle install (if you added bower assets)
  • bower install (if you added gems)
  • Restart your web server (if you added runtime things or gems)
  • Go to /some_page in your browser and do X, Y and Z (if there is a visual components)
  • Open up your rails console and type some code (if there are code components to test)

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