remove hardcoding and move those values to constants