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

Consolidate duplicated models #5076

Closed
viddo opened this issue Aug 18, 2015 · 2 comments
Closed

Consolidate duplicated models #5076

viddo opened this issue Aug 18, 2015 · 2 comments

Comments

@viddo
Copy link
Contributor

viddo commented Aug 18, 2015

Right now we some models for the same imports endpoint:

Also on the subject of models, IMHO, we would be better of to keep all models that represent a concept/API endpoint in one place, e.g. models/ (since that seems to be the commonplace). @CartoDB/frontend what do you think about moving them to that directory than have them spread out in various locations (e.g. move from common/background_polling to models/?

@viddo viddo added the Frontend label Aug 18, 2015
@viddo viddo self-assigned this Aug 18, 2015
@viddo viddo added this to the Torrelodones milestone Aug 18, 2015
@xavijam
Copy link
Contributor

xavijam commented Aug 18, 2015

Fine by me 👍

@viddo viddo changed the title Remove duplicated import model Remove duplicated models Aug 18, 2015
@viddo viddo changed the title Remove duplicated models Consolidate duplicated models Aug 18, 2015
@viddo viddo modified the milestone: Horcajo de Santiago Oct 7, 2015
@alonsogarciapablo
Copy link
Contributor

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.

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

No branches or pull requests

3 participants