we should probably create a set of constants for these to replace the magic strings