SalesforceLabs / survey-force

85 stars 61 forks source link

Additional field cleanup and permissions additions #79

Closed dschach closed 4 years ago

dschach commented 4 years ago

Found fields that are not used. Added permission set.

jrattanpal commented 4 years ago

@dschach

Thanks for these change. Unfortunately, I didn't check all the tests before I merged into master. So master branch is not failing and I cannot promote this to packaging org for new package version for AppExchange.

ViewSurveyController_Test is the only class failing. I know you made many changes there so you are familiar with those. Can you please look into this?

Log file is attached:: apex-07LJ000000u61tiMAA.log

Thanks

jrattanpal commented 4 years ago

Nvm. I figured it out. I wasn't using it correctly when creating standard controller.

https://github.com/SalesforceLabs/survey-force/commit/dba9295c1f558dfb4da8df97d3ecec7a6293abc9