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

update to spring boot 3 #1258

Open
fer-marino opened this issue Feb 5, 2024 · 10 comments
Open

update to spring boot 3 #1258

fer-marino opened this issue Feb 5, 2024 · 10 comments
Assignees
Labels
enhancement New feature or request

Comments

@fer-marino
Copy link

I've noticed that this project still uses spring 2.7, which unfortunately has been discontinued. Would be nice to rebase on spring boot 3!

Just forcing the dependency doesn't seems to work unfortunately. Thanks for your work

@straumat straumat self-assigned this Feb 8, 2024
@straumat
Copy link
Member

straumat commented Feb 8, 2024

@fer-marino thanks for your issue. I will try to take time to do it!

@straumat straumat added the enhancement New feature or request label Feb 8, 2024
@xerikssonx
Copy link
Contributor

"Hi, thanks for all your hard work on Cassandre—it's been incredibly helpful! I was wondering if you have any plans to update it to Spring Boot 3, as I'd be interested in using the new features."

@straumat
Copy link
Member

HI @xerikssonx and thanks, i will try to find some time in december

@xerikssonx
Copy link
Contributor

@straumat could you please allow me create branches and pull requests, I working on migration to Spring boot 3

@straumat
Copy link
Member

straumat commented Nov 4, 2024

 @xerikssonx Hi ! I think you can create a fork and ask for me to merge your work once you are done ?

@xerikssonx
Copy link
Contributor

@straumat done
#1310

@straumat
Copy link
Member

straumat commented Nov 6, 2024

@xerikssonx Thanks ! I try to review it this week. Just one question : why do you remove ZonedDateTime ?

@xerikssonx
Copy link
Contributor

@straumat it did not work with ZonedDateTime. But I check fix that I've added for OffsetDateTime works also with ZonedDateTime. So I reverted changes to ZoneDateTime back. Could you pleasetrigger build again?
Also pay attantion, I change expecting in one test to code 401 from 403 (authentication.test.js).

@xerikssonx
Copy link
Contributor

@straumat hi, are you going to create new release with spring boot 3?

@straumat
Copy link
Member

@xerikssonx Hi ! Yes, but i need to fix some issues with my key :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants