-
Notifications
You must be signed in to change notification settings - Fork 47
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
Library: Use new columns #2151
Library: Use new columns #2151
Conversation
src/Abstractions/NexusMods.Abstractions.NexusWebApi/Types/V2/GameId.cs
Outdated
Show resolved
Hide resolved
This PR conflicts with |
9796ea1
to
4bd5e6c
Compare
This PR doesn't conflict with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, I ran into the
[ERROR] While commiting transaction|System.InvalidOperationException: Invalid UTF-8 sequence.
bug when testing mass deletions etc, that should be solved by #2154.
Would like to get that in first so this can be tested better.
5236d21
to
1a02072
Compare
This PR conflicts with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested now with the fix from #2154 and it appears to be working.
Another pr seems to have been merged in the meantime causing another merge conflict but this PR should be good
1a02072
to
9acf06f
Compare
This PR doesn't conflict with |
Due to the composition aspect of the new system, there is quite the amount of boilerplate code.
TODO: