CL Fisher
CL Fisher
@Learningstuff98 I'm going to try to look at this today.
@Learningstuff98 I'd like to see "No Social Media Presence" treated the same way as other checkboxes on the profile form -- for clarity: with the prompt "No Social Media Presence"...
@Learningstuff98 That looks like it should be right . Please also address "**** you should not be able to successfully submit for approval with the media information in this state...
@Learningstuff98 Also make it so that when the bank views the partner, they see all the media fields [see http://localhost:3000/diaper_bank/partners/1 , signed in as [email protected]]
@Learningstuff98 Will do, but probably tomorrow at this point.
@Learningstuff98 Hi! I'll add separate comments for each thing I find from testing from now on. 1/ The banks view of the partner still does not show the new fields:...
@Learningstuff98 2/ The partner can submit for approval with an entirely blank media section. They shouldn't be able to.
@Learningstuff98 Argh. I don't think I pulled it down again. Morning brain. Will retest.
@Learningstuff98 #1 passes. #2 doesn't. However, that requirement was not in the issue as described.
@Learningstuff98 Have created the issue and assigned it to you. Thanks!