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

Let us touch submodels and supermodels #3

Closed
wants to merge 5 commits into from

Conversation

dezmathio
Copy link

I've seen that in README you suggest that attributes that are updated via time/touch should live on the supermodel, however, our legacy infrastructure has some events that live on the submodel. I was wondering if we could coexist?

image

@coveralls
Copy link

coveralls commented Apr 10, 2017

Coverage Status

Coverage increased (+0.01%) to 99.301% when pulling a46e8df on dezmathio:touch_fix into e82011b on krautcomputing:master.

1 similar comment
@coveralls
Copy link

coveralls commented Apr 10, 2017

Coverage Status

Coverage increased (+0.01%) to 99.301% when pulling a46e8df on dezmathio:touch_fix into e82011b on krautcomputing:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 99.304% when pulling 80e430b on dezmathio:touch_fix into e82011b on krautcomputing:master.

1 similar comment
@coveralls
Copy link

coveralls commented Apr 11, 2017

Coverage Status

Coverage increased (+0.01%) to 99.304% when pulling 80e430b on dezmathio:touch_fix into e82011b on krautcomputing:master.

@manuelmeurer
Copy link
Collaborator

You got it, buster!

Implemented in b780d8f

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

Successfully merging this pull request may close these issues.

3 participants