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

Feature/jakarta lift #100

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

tuxBurner
Copy link

Hello,
this is a Lift from javax to jakarta.
There where several Changes but it is running and works.
The Version is bumped to 2.0.0-SNAPSHOT.

Would you consider to release this version in the maven repository ?

Greetings

tuxBurner and others added 14 commits February 16, 2023 10:43
- updated some libraries (spring, log4j, jakarta.mail, jackson ...)
- restored configurations and configurationStash since static configuration leads to overwriting configurations for multiple StripesFilter (plugins)
- added log4j.xml and remoced old log4j.properties
- merged local changes of Mock-classes to stripes Github ... e.g. require ServletContext for new Thymleaf version, added JavaDocs + @Override-annotations and so on
…eir contexts (configurations in Stripesfilter not unambiguous)
@rgrashel
Copy link
Member

rgrashel commented Oct 26, 2023 via email

@tuxBurner
Copy link
Author

Nice @rgrashel if you need any support, let me know ;)

@herrlado
Copy link

Any news? This branch is java 17 compatible and has the newest libraries (spring 6.x)

@jellisgwn
Copy link

Is there any progress on getting this new version push to a repo?

@herrlado
Copy link

@rgrashel Hey Rick, could you make a statement on this pr ? We do need this for several project. We are considering to fork it and maintain on our own if nothing happens (we do not need just jakarta but also newest spring etc). We would prefer to support stripes in main development but we need some feedback...

@elijah-potter
Copy link

Are there any updates on this? @herrlado, did you end up forking it?

@tuxBurner
Copy link
Author

@elijah-potter we are currently working with our fork and have it running in production.

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

Successfully merging this pull request may close these issues.

6 participants