Skip to content

Trexle Payment Gateway#2

Open
jyotigautam wants to merge 22 commits intodevfrom
trexle
Open

Trexle Payment Gateway#2
jyotigautam wants to merge 22 commits intodevfrom
trexle

Conversation

@jyotigautam
Copy link
Member

No description provided.

@jyotigautam jyotigautam changed the title Trexle Trexle Payment Gateway Dec 21, 2017
@pkrawat1
Copy link
Member

pkrawat1 commented Jan 4, 2018

@jyotigautam what is the status of this branch?

defmodule Gringotts.Examples.Trexle do

alias Gringotts, as: Billing
alias Billing.{CreditCard, Address, Worker, Gateways}
Copy link
Member

Choose a reason for hiding this comment

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

remove alias for Worker and Gateways as this is not been used.

Copy link
Member

@chandradot99 chandradot99 left a comment

Choose a reason for hiding this comment

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

LGTM 🥇

Copy link
Member

@gopalshimpi gopalshimpi left a comment

Choose a reason for hiding this comment

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

Looks Good To Me 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants