Should the dependency array here be empty?

Hey sorry, I'm checking out. if you use empty you will run getAllMessages once. If you desire to rerun that function anytime there is a change you need a variable that gets toggled when the .on call triggers ( a change so reload) If the thing in [] changes you rerun the function. So it depends on what you are trying to do with the subscription, if anything at this point in your testing.