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

Fixed typo #1

Merged
merged 1 commit into from
Mar 4, 2014
Merged

Fixed typo #1

merged 1 commit into from
Mar 4, 2014

Conversation

hpig
Copy link
Contributor

@hpig hpig commented Mar 4, 2014

No description provided.

Willianvdv added a commit that referenced this pull request Mar 4, 2014
@Willianvdv Willianvdv merged commit 3219ccc into Willianvdv:master Mar 4, 2014
@Willianvdv
Copy link
Owner

Thanks!

@hpig
Copy link
Contributor Author

hpig commented Mar 4, 2014

No problem :)

Ik heb een vraagje over dit stukje in de readme:

Create a payment method with Spree::BillingIntegration::Omnikassa as provider.

Hoe creëer ik deze? Ik kan hem in /admin/payment_methods/new niet selecteren uit het lijstje van methods en het word me uit de spree docs ook niet duidelijk.

Groeten,
Huib

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4ece4e5 on hpig:patch-1 into cc6a1da on Willianvdv:master.

@Willianvdv
Copy link
Owner

Welke Spree versie gebruik je? Ik heb deze wel 2-2 compatible gemaakt maar nog niet gebruikt op 2-2. Vanmiddag ga ik er toevallig mee aan de slag, misschien zijn de betaalmethodes veranderd waardoor Omnikassa nu niet te selecteren is.

Ik houd je op de hoogte!

@hpig
Copy link
Contributor Author

hpig commented Mar 5, 2014

Ik zit op 2.1.4. Tot welke versie zou het moeten werken?

@Willianvdv
Copy link
Owner

2-1 :)

@Willianvdv
Copy link
Owner

@hpig Lost #2 jou probleem op?

@hpig
Copy link
Contributor Author

hpig commented Mar 27, 2014

Ja dit werkt, bedankt! Nu wel andere problemen bij de checkout (die wel werkte voor de install van spree-omnikassa), maar dat zal ik nog nader onderzoeken om met een wat gerichtere omschrijving te komen.

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.

None yet

3 participants