don't hardcode url and make it a parameter