Nick Airdo

Looks good @cabal95. I'm looking forward to running this later tonight. I'm pretty sure we've got the storage resource you described above, although I'd like to change the folder/file name...

@cabal95 No, not quite -- but perhaps partially. But we are about to refactor all the existing `Rock.Tests` from Xunit to MS Test, remove/move any db-context required tests into `Rock.Tests.Integration`...

@cabal95 Beautiful. Your changes are also working in my environment and I'm integrating them into our refactored `feature-dl-update-test-projects` branch via e282b1cc50054c1a3f92441557f260fc3b6b60cc.

@pauldavidmoran or @Tschrock If either of you can confirm you are still seeing this on your v9.2 or greater instance can you kindly re-open this (or let me know).

Jennifer, thanks for the update. We're going to look into this one again.

@jennifertroeger Can you give us a few more details including: - What web browser are you using? - Browser's version? - OS version? - anything else you think might be...

Hi @nlBayside , I looked at your change and it might be better if this `...GetEditValues( metric )` and its `SaveAttributeValues()` was moved to _after_ the `if ( metric.Id ==...

Thanks @nlBayside We typically get to pull requests depending on our workload. As far as signing a the [Spark Contributor Agreement](http://www.rockrms.com/Content/RockExternal/Misc/Contributor%20Agreement.pdf), that's always a good idea --but submitting a Pull...

@cabal95 Thanks. Did you do that test on the pre-alpha-release branch, develop, or other?

@cabal95 Ok, yeah, it was working fine in v6. I only started seeing this once we pulled from pre-alpha-release (1.7.0.39).