Make grade_id into grade_uuid #2208
Assignees
Labels
Comments
|
I'll take this one. |
jniles
added a commit
to jniles/bhima
that referenced
this issue
Oct 19, 2017
This commit renames the "grade_id" column to "grade_uuid" in reflection of it's underlying uuid representation. The two commands that were run were: ```sh find ./client/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./server/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' ``` Closes Third-Culture-Software#2208.
jniles
added a commit
to jniles/bhima
that referenced
this issue
Oct 19, 2017
This commit renames the "grade_id" column to "grade_uuid" in reflection of it's underlying uuid representation. The three commands that were run were: ```sh find ./client/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./server/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./test/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' ``` Closes Third-Culture-Software#2208.
This was referenced Oct 19, 2017
jniles
added a commit
to jniles/bhima
that referenced
this issue
Oct 23, 2017
This commit renames the "grade_id" column to "grade_uuid" in reflection of it's underlying uuid representation. The three commands that were run were: ```sh find ./client/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./server/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./test/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' ``` Closes Third-Culture-Software#2208.
jniles
added a commit
to jniles/bhima
that referenced
this issue
Oct 24, 2017
This commit renames the "grade_id" column to "grade_uuid" in reflection of it's underlying uuid representation. The three commands that were run were: ```sh find ./client/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./server/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./test/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' ``` Closes Third-Culture-Software#2208.
bors bot
added a commit
that referenced
this issue
Oct 24, 2017
2216: fix(employee): rename grade_id to grade_uuid r=lomamech a=jniles This commit renames the "grade_id" column to "grade_uuid" in reflection of it's underlying uuid representation. The three commands that were run were: ```sh find ./client/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./server/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' find ./test/ -type f | xargs sed -i 's/grade_id/grade_uuid/g' ``` Partially addressed #342. Closes #2208.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See the discussion on #2191 (comment)
The employee table has a column called
grade_uuidthat references the columngrade.uuid. It is quite confusing. All uuid columns should be suffixed with_uuidto help programmers know that they are dealing with a binary uuid on the underlying table.Someone should change this column to be
grade_uuidand update the associated tests.The text was updated successfully, but these errors were encountered: