Make sample model for admin/accounts singular rather than plural. #733

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@thomas-holmes
RSpec member
@thomas-holmes
RSpec member

Is it really this simple?

@thomas-holmes
RSpec member

I see now that various configurations are failing for admin/accounts related reasons.

Looks like it's something related to the master branch of rails.

@alindeman

Huh! Maybe there is something more to do this. Thanks for the pull request: I'll use your code to dig a bit more.

@thomas-holmes
RSpec member
@alindeman

If you look at the master build before my pull request you see failures in
widget stuff in all versions. CI Builds using rails master have additional
failures.

You're right. This is my fault. I'll fix it shortly.

@alindeman alindeman added a commit that closed this pull request Apr 7, 2013
@alindeman alindeman Merge branch 'plural-accounts-fix'
[Closes #733] [Fixes #731]
5ee4fe6
@alindeman alindeman closed this in 5ee4fe6 Apr 7, 2013
@alindeman

Once I fixed my failures (I blame trying to code while watching the Final Four and it being late), your fix worked great. I rebased and merged it in.

Thanks a lot 🌈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment