Skip to content

Store normandy id fixes #910#1073

Merged
jaredlockhart merged 1 commit into
masterfrom
910
Mar 20, 2019
Merged

Store normandy id fixes #910#1073
jaredlockhart merged 1 commit into
masterfrom
910

Conversation

@jaredlockhart
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't look right. Did this get left out of a previous PR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It suuuuurrreeeee did. #1068

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should add a step to continuous integration to check that no migrations got missed.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Filed #1079

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should, yeah. We did it on Normandy recently, and it was a pretty easy check.

Do you want to leave this here and merge as is, or do you want to do something to fix this?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well It has to go in and I don't see the point of splitting it into its own tiny PR so I'm fine with this as is.

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.

2 participants