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

Possible issue causing problem with marksman? #3

Open
ReticentRobot opened this issue Mar 19, 2021 · 0 comments
Open

Possible issue causing problem with marksman? #3

ReticentRobot opened this issue Mar 19, 2021 · 0 comments

Comments

@ReticentRobot
Copy link

Hello,

There are several people (self included) that are running into problems using marksman with Snipe-IT version 5+. I think I've traced the problem to line 45 in Category.cs in your fork of SnipeSharp:

public bool eula { get; set; }

I've found that if I change it to:

public string eula { get; set; }

and then build the SnipeSharp and then marksman projects, that this gets marksman working again on Snipe-IT 5+. Would you be able to verify that (I'm not a developer, and have no idea if this will cause problems or if changing from bool to string would be the right variable type), and if so - implement a fix to get it working again?

Thanks so much!!

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

No branches or pull requests

1 participant