Skip to content
This repository has been archived by the owner on Jul 8, 2020. It is now read-only.

route count includes disabled routes #125

Open
skeltont opened this issue Jul 25, 2016 · 4 comments
Open

route count includes disabled routes #125

skeltont opened this issue Jul 25, 2016 · 4 comments
Milestone

Comments

@skeltont
Copy link
Collaborator

skeltont commented Jul 25, 2016

on the admin/users listing it shows how many routes the user has. it should only count routes that are available, but seeing that this number is stored in the db that's kind of tough.

@jonahgeorge jonahgeorge added this to the Alpha milestone Jul 25, 2016
@jonahgeorge
Copy link
Member

This means either we do multiple counter_caches. Ie: User#active_routes,User#expired_routes.

If not, we need to drop that metric from the #index route. It will be too computationally expensive to compute for every user.

@skeltont
Copy link
Collaborator Author

I think we'd be fine to drop it, honestly. It's p expensive for something not super relevant. quantity over quantity.

@jonahgeorge
Copy link
Member

"quantity over quantity"

@skeltont
Copy link
Collaborator Author

skeltont commented Aug 1, 2016

"quantity over quantity"

damn

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants