Remove Profile feature flag and its references

[abroddrick]

Issue description

We have a lot of checks in our code checking for if the profile feature flag is on. That said this feature has been on for a while and we seem to have no complaints. Moving forward any issue we have we should just have a ticket for as a bug or new addition to the profile feature, but we do not need to turn off the feature entirely. Thus there is no longer a reason to keep the flag or it's references. We can clean that up.

Acceptance criteria

• Remove all references to the profile feature flag including if checks
• fix related unit tests and remove any obsolete ones if found

Additional context

Note: this really should be about a 2, but buffering this with 3 points as removing stuff can end in more unit tests breaking than we expect.

Links to other issues

blocked by #2162