search

Dwolla / dwolla-v2-csharp

Official C# Wrapper for Dwolla's API
https://developers.dwolla.com
MIT License
12 stars 16 forks source link

Add Business Classification models #2

Closed ithielnor closed 7 years ago

ithielnor commented 7 years ago

Also added some basic UI to the example app.

therockstorm commented 7 years ago

Thanks for the PR, sorry for not merging directly. I made changes and pushed them to master since there were a few other cleanup items.

ithielnor commented 7 years ago

No problem. Do you have any style preferences you prefer? I noticed some of your changes were just style, and I'll happily comply with any guidelines you supply for future PRs.

therockstorm commented 7 years ago

We don't have a published style guide so it's mostly personal preference. We do use JetBrains Rider for our IDE so use its Format Code option and some of its refactoring suggestions. I wouldn't decline a PR based purely on style so long as it's mostly in line with the existing code.