valentinajemuovic / banking-kata-java

Banking Kata (Java)
MIT License
346 stars 92 forks source link

#86 Adapter - Persistence - Redis #89

Closed JoaoCipriano closed 1 year ago

JoaoCipriano commented 1 year ago

Hello Valentina! I saw the post on LinkedIn about this project and issue #86 matches my studies about Redis and my experience. Actually, I’ve never worked with Redis using Spring Data only caching objects, retrieving and invalidating direct with an external dependency library.

I learned a lot about this issue and hope that be useful, follow the additions and changes:

valentinajemuovic commented 1 year ago

@JoaoCipriano, great work! I merged the PR.

As a follow-up, I did a review with some suggestions about the test approach, you can see here #90.

JoaoCipriano commented 1 year ago

Thank you @valentinacupac! I'll check the suggestions.