softwerkab / fortnox-csharp-api-sdk

.NET SDK for Fortnox API.
MIT License
52 stars 64 forks source link

Add static class with all predefined accounts #285

Closed bpaolo71 closed 8 months ago

bpaolo71 commented 8 months ago

It should work now

richardrandak commented 8 months ago

Could you rename the folder from "Predefined Accounts" to something like "Constants"?

richardrandak commented 8 months ago

Sorry, I see the folder was already there :) ..I just don't like the namespace with underscore. Can you please remove the "Predefined_Accounts" part and let it simply reside in Entities namespace?

bpaolo71 commented 8 months ago

Of course. I didn't realize this before you told me :)

bpaolo71 commented 8 months ago

It looks like there are tests that fail but they are not related to this specific pull request.