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

Fix and update swagger client generation #384

Merged
merged 5 commits into from
Oct 27, 2024

Conversation

cyb3rko
Copy link
Contributor

@cyb3rko cyb3rko commented Oct 18, 2024

Hey again,

I wanted to tackle #372 but it showed the following error:

It seems that after years of warnings the time of updating the client has come.


So I generated a new client version (including Java 11 instead of Java 7). It needed some adjustments (see new README instructions and updated gradle code), but it works now.

I've used the newest available version of Swagger Codegen.

What do you think? There shouldn't be breaking changes in the code itself, right?
I will test the new version some more, but for now everything seems to be working.

@jmattheis
Copy link
Member

Sorry, I'm currently a little busy, I'll review/test this next week.

@jmattheis jmattheis self-requested a review October 20, 2024 21:05
Copy link
Member

@jmattheis jmattheis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jmattheis jmattheis merged commit b4474a2 into gotify:master Oct 27, 2024
2 checks passed
@cyb3rko cyb3rko deleted the java-client-update branch October 27, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants