Skip to content

make company model swappable#30

Merged
walterrenner merged 1 commit intoarteria:masterfrom
imposeren:use-swappable-model
May 22, 2018
Merged

make company model swappable#30
walterrenner merged 1 commit intoarteria:masterfrom
imposeren:use-swappable-model

Conversation

@imposeren
Copy link
Contributor

@imposeren imposeren commented Jan 22, 2018

Company model should be swappable. This should fix:
#25 #28

Also when custom model is used TestRunner is unable to serialize data for openninghours.Company:

    return self.cursor.execute(sql, params)
django.db.utils.ProgrammingError: relation "openinghours_company" does not exist
LINE 1: ...mpany"."slug", "openinghours_company"."logo" FROM "openingho...

This MR should fix this too

@coveralls
Copy link

coveralls commented Jan 22, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.05%) to 85.531% when pulling 00587e1 on imposeren:use-swappable-model into 0a80b43 on arteria:master.

@imposeren imposeren mentioned this pull request Jan 22, 2018
Open
@SirEdvin
Copy link

SirEdvin commented Mar 6, 2018

@arteria, can you please merge it?

@imposeren
Copy link
Contributor Author

imposeren commented May 21, 2018 •
edited
Loading

@jvamvas, @walterrenner, can anyone from arteria please merge this and release new version on PyPI?

@walterrenner walterrenner merged commit ef7626a into arteria:master May 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@imposeren @coveralls @SirEdvin @walterrenner