redis-om-python icon indicating copy to clipboard operation
redis-om-python copied to clipboard

"getting_started.md" doc requires some updates

Open oren-twigo opened this issue 3 years ago • 1 comments

The getting_started.md requires some fixes:

  1. In the example of: from redis_om import get_redis_connection redis_conn = get_redis_conection()

Should be updated to: from redis_om import connections redis_conn = connections.get_redis_connection()

This fix is required also in the following docs: https://github.com/redis/redis-om-python/blob/main/docs/connections.md https://github.com/redis/redis-om-python/blob/main/docs/models.md

  1. a small typo: replace "...Here, we run JSON.GET command with.." with "...Here, we run HGETALL command with..."

  2. It is better to add some link or duplicate the following article into this repo docs: https://redis.io/docs/stack/get-started/tutorials/stack-python/

  3. in https://github.com/redis/redis-om-python/blob/main/docs/models.md the description of "primary_key_pattern" setting is not clear enough. Missing an example for anything but the default "{pk}" value.

  4. it is better to show an example the auto-generated pk (ULID) can be simple overridden with any other value before the "save" command. This allows to create keys which are composed of other fields' values, and not a single UUID. for example: for relation "table" between two models.

oren-twigo avatar May 05 '22 02:05 oren-twigo

Thanks - I'll take a look as able... if you'd like to create a PR for (2) you can become a contributor!

simonprickett avatar May 06 '22 12:05 simonprickett