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

fix(employee): rename grade_id to grade_uuid #2216

Merged

Conversation

jniles
Copy link
Collaborator

@jniles jniles commented 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:

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.

@jniles jniles requested a review from lomamech October 19, 2017 08:09
@jniles
Copy link
Collaborator Author

jniles commented Oct 19, 2017

@lomamech, could I get a review?

@jniles jniles force-pushed the chore-make-grade_id-into-grade_uuid branch from cf2056f to 13d67df Compare October 23, 2017 11:25
@jniles jniles added the Bug Fix label Oct 24, 2017
@lomamech
Copy link
Collaborator

@jniles The Changes on this PR are not a problem, please update this PR is outdated

@jniles
Copy link
Collaborator Author

jniles commented Oct 24, 2017

@lomamech, thank you for the review!

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 jniles force-pushed the chore-make-grade_id-into-grade_uuid branch from 13d67df to af89a6d Compare October 24, 2017 09:23
@jniles
Copy link
Collaborator Author

jniles commented Oct 24, 2017

@lomamech, I have updated the branch. If you type "bors r+", it should start the merge process.

@lomamech
Copy link
Collaborator

bors r+

bors bot added a commit that referenced this pull request 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.
@bors
Copy link
Contributor

bors bot commented Oct 24, 2017

Build succeeded

@bors bors bot merged commit af89a6d into Third-Culture-Software:master Oct 24, 2017
@jniles jniles deleted the chore-make-grade_id-into-grade_uuid branch October 24, 2017 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants