redis-developer / redis-developer.github.io

The Home of Redis Developers
https://developer.redis.com
47 stars 142 forks source link

index-redis-om-spring-hash.mdx is outdated #451

Open TianMing2018 opened 2 years ago

TianMing2018 commented 2 years ago

this chapter has few errors and need to updated for better learning

  1. groupId is wrong, it should be com.redis.om now
    <dependency>
    <groupId>com.redis.om.spring</groupId>
    <artifactId>redis-om-spring</artifactId>
    <version>0.1.0-SNAPSHOT</version>
    </dependency>
  2. nethier release version 0.6.0 nor latest snapshot 0.5.2-SNAPSHOT will create UserIdx index

    it seems that not so 'distinguish' to tell which om version reference to which major version of spring-boot. assume that's probles, i have upgrade spring-boot to 2.7.2 and use redis-om-spring:0.6.0 but it still won't create UserIdx

By the way,what's difference betwen https://github.com/redis-developer/redis-om-spring-skeleton-app and this chapter?

TianMing2018 commented 2 years ago

it seems that you docs now is the only version works ,i'm wondering what should we do if we want to use spring-boot 2.6.x or spring-boot-2.7.x ,which redis-om-spring version should we pick? do this chapter still in developing or discarded now

wjohnsto commented 2 years ago

@bsbodden I updated the pom.xml file information for the relevant docs for this issue, but I am not sure about the UserIdx question or the Spring Boot version questions. Can you update the docs to reflect what is current?

The relevant docs are here

TianMing2018 commented 2 years ago

you can see the detial about this issuse on url https://github.com/TianMing2018/redis-developer.github.io/blob/master/docs/develop/java/spring/redis-om/index-redis-om-spring-hash.mdx

the spring-boot version problem is a small confusion when i first read this chapter and found this problem,it seems that i have seen userIdx been created when i change spring-boot and redis-om-spring version, so i'm confused about which redis-om-spring should use which spring-boot version . can't remember the detail.