Use FeatureStoreId instead of String

This uses the temporary `OldIdentifier` until we migrate to `Identifier`
Didn't you add the macro things with OldIdentifier to prevent the need for the unsafe get, i.e. FeatureStoreId(Identifier("00000")) should compile and be safe?