It's actually wrong to turn it off because javascript: is like eval for the browser, and eval is evil, as people know.
Alternatively, you do not need to put an action prop. preventDefault in your join method. For instance:
function join(e){ e.preventDefault();
source share