-
Notifications
You must be signed in to change notification settings - Fork 85
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
Feedback #261
Comments
Thank you for your feedback. I think it will be great if you will open a pull request with all of your suggestions in order to improve the library. |
Hi @zapadi, I think my interpretation of the library would be a mess of breaking changes, so users wouldn't be happy about it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
thanks for maintaining this project.
I'm using your project in a small bridge project between different systems.
Maybe you want to improve the usability of the library, so here's my feedback after working with it for two days:
TimeEntry
, so I'm loading the requiredTimeEntryActivity
for theActivity
property. But wait, I can`t assign my retreived activity since there are some type shenanigans going on. Related to Version 3.0.6.1 makes IssueCustomField.Info readonly breaking existing usage #253GetObjects<T>()
-method (without theNameValueCollection
parameter. Or even better, create individual services per type, to support individual queries (GetProjectsByIdOrIdentifierAsync(string id)
for example).ISerializer
interface and implement it per XML and JSON? Because of the limited object (de)serialization capabilities, I cannot serialize the object by myself to cache them. For now, I sticked with custom dtos to make them cacheable properly.Redmine.Net.Issues.Issue
instead ofRedmine.Net.Types.Issue
(i'm thinking about something likeRedmine.Net.Issues.IRedmineIssueClient
, so you have all related models in the same namespace.The text was updated successfully, but these errors were encountered: