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

Application upgradation #72

Open
cooligc opened this issue Apr 16, 2018 · 12 comments
Open

Application upgradation #72

cooligc opened this issue Apr 16, 2018 · 12 comments

Comments

@cooligc
Copy link

cooligc commented Apr 16, 2018

Hi,

Is there any plan to upgrade spring boot to 2.0.1.RELEASE version ? If yes, when it will be available tentatively ?

@robertotru
Copy link
Member

@cooligc not so soon I guess. I'm in rush with other projects, I don't have time to dedicate to the upgrade.

Indeed, this is an open source project. I'll accept contributions.

@unaor
Copy link

unaor commented May 17, 2018

watching

@robertotru
Copy link
Member

I think that in the next days I may find the right mood for supporting Spring Boot 2.x in this project.

@robertotru
Copy link
Member

I've started the upgrade. First thing I'm trying to accomplish, is to move the default services for REDIS persistence in a separate module. Since email persistence is not mandatory, I think that putting the default service there was a very bad idea.
I'm not sure how much time it will take the migration. There are dozens of broken tests and changes in SpringBoot2 that must be taken into account.

@limpep
Copy link

limpep commented Jul 27, 2018

Just was wondering if there was any progress on this? Thanks.

@robertotru
Copy link
Member

@limpep I have a lot of issues, and every time I find a new one I lose interest. But I'm working on it. Unfortunately, I don't have a release date at the moment.

Wanna help?

@limpep
Copy link

limpep commented Aug 6, 2018

@robertotru I would love to help but I don't know where to begin plus I think I'll probably mess it up more than actually helping 😂

@robertotru
Copy link
Member

I'm thinking about getting rid of the REDIS support (if one wants to have email persistence, can implement the service he prefers and also all the template engine but not Mustache.

This may speedup the upgrade.

Opinions?

@cldsnchz
Copy link

cldsnchz commented Aug 9, 2018

I think it makes total sense to move the PersistenceService implementation (redis) to its own module.
If having that module is a pain to maintain then remove it, but it’s good thing to have a reference implementation.

Regarding the templates, I guess that having one reference implementation it’s enough, mostly because the integration is very similar among those engines.

It is nice to have those integrations done, but if having them means that you don't have time (or motivation) to maintain the core, in that case just get rid of them.

@robertotru
Copy link
Member

@rezaep
Copy link

rezaep commented Nov 27, 2018

Please indicate the compatible Spring boot version in the home page of the repository. It can save time from other developers who want to integrate this library into Spring boot 2 projects. I didn't see anything about this issue and started to integrate this library into my project and while running I got the error and after searching for the reason I have found this open issue.

@robertotru
Copy link
Member

@rezaep it is on the top banner, but maybe it should be also in the readme. We also do accept PRs :)

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants