massenergize / frontend-portal

Front End Code for the MassEnergize Community Portal. The community portal is customized for each participating community, and has the canonical URL https://community.massenergize.org/<community-name>/. For example, https://community.massenergize.org/wayland/.
MIT License
5 stars 3 forks source link

DEV: leave testimonials: can't leave photo #337

Closed kaatvds closed 4 years ago

kaatvds commented 4 years ago

when I click on "choose an image" the button goes white and nothing happens

BradHN1 commented 4 years ago

From javascript console:

TypeError: Cannot read property 'click' of null at a.value (StoryForm.js:98) at onClick (StoryForm.js:240) at Object.s (react-dom.production.min.js:14) at d (react-dom.production.min.js:14) at react-dom.production.min.js:14 at m (react-dom.production.min.js:15) at ae (react-dom.production.min.js:52) at ie (react-dom.production.min.js:51) at se (react-dom.production.min.js:52) at de (react-dom.production.min.js:56) at N (react-dom.production.min.js:287) at H (react-dom.production.min.js:19) at Je (react-dom.production.min.js:70) at Ze (react-dom.production.min.js:69) at e.unstable_runWithPriority (scheduler.production.min.js:19) at Yi (react-dom.production.min.js:122) at D (react-dom.production.min.js:287) at $e (react-dom.production.min.js:68) at HTMLDocument.r (helpers.ts:87)

frimpongopoku commented 4 years ago

Hi @kaatvds & @BradHN1 , so I am testing this ticket now, and it looks like its one of the bugs that came up because we werent fully switched to Sam's new system and my new components yet. I have been playing with the StoryForm image chooser, and all is well on my side. Can I get a scenario of exactly what you did to get the error? Cos all is working over here. In the mean time, I will move this to back log 🥂

BradHN1 commented 4 years ago

@frimpong I’ve already worked on testimonials and tested that works. I thought I had moved it to Deployed in Dev, where it should be.

On Sep 28, 2020, at 1:10 PM, Frimpong Opoku Agyemang notifications@github.com wrote:

Hi @kaatvds https://github.com/kaatvds & @BradHN1 https://github.com/BradHN1 , so I am testing this ticket now, and it looks like its one of the bugs that came up because we werent fully switched to Sam's new system and my new components yet. I have been playing with the StoryForm image chooser, and all is well on my side. Can I get a scenario of exactly what you did to get the error? Cos all is working over here. In the mean time, I will move this to back log 🥂

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/massenergize/frontend-portal/issues/337#issuecomment-700165858, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADBVDZBIOJYYWSALLWOZOZ3SIC7QLANCNFSM4RAN2K3A.

frimpongopoku commented 4 years ago

Nice, I will move it 🎊

kaatvds commented 4 years ago

I filled in form and added a photo. Clicked submit. took a split second to react, then said "we are sending now" where it got stuck in loop image

frimpongopoku commented 4 years ago

I am pretty sure that isn't an infinite loop 🤣 🤣 . That notice is meant to be there, until your testimonial goes through, then you will get another one notifying you that all is well, or a red one, telling you something went wrong. I guess you testimonial took longer to go through and made you think the notice was stuck because you probably selected a large image. The form works, I tested again now.

When there aren't any more pressing bugs to fix, I will make a component that forces users to crop and resize images before they are even able to submit. This way, uploads will be faster, and the experience will be smoother.

kaatvds commented 4 years ago

Hi Frimpong, it circled for over 10 minutes, and then I closed the tab. I also don't see the testimonial in my list as a Sadmin. Kaat

On Tue, Sep 29, 2020 at 2:55 PM Frimpong Opoku Agyemang < notifications@github.com> wrote:

I am pretty sure that isn't an infinite loop 🤣 🤣 . That notice is meant to be there, until your testimonial goes through, then you will get another one notifying you that all is well, or a red one, telling you something went wrong. I guess you testimonial took longer to go through and made you think the notice was stuck because you probably selected a large image. The form works, I tested again now.

When there aren't any more pressing bugs to fix, I will make a component that forces users to crop and resize images before they are even able to submit. This way, uploads will be faster, and the experience will be smoother.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/massenergize/frontend-portal/issues/337#issuecomment-700914809, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHHAB3HCRZWN32DQ27F3Y3DSIIURBANCNFSM4RAN2K3A .

-- -- Kaat Vander Straeten, Ph.D.

"Look, Ma, no hands!" "“Ring the bells that still can ring / Forget your perfect offering / There is a crack / A crack in everything / That’s how the light gets in” (Leonard Cohen)

Co-founder and Vice President of MassEnergize http://www.massenergize.org/, a 501(c)3 | Coordinator of Energize Wayland http://www.energizewayland.org/ | Associate Member of the Wayland Energy and Climate Committee https://www.wayland.ma.us/Pages/WaylandMA_BComm/Energy/index (an MCAN https://www.massclimateaction.org/Chapter) | Co-founder of Transition Wayland http://transitionwayland.org/ | Certified Permaculture Designer (Sowing Solutions https://sowingsolutions.org/upcoming-courses/) | Graduate of E. Ingham's Life in the Soil classes (Environment Celebration https://environmentcelebration.com/) | Beekeeper, consultant, and founder of BEElieve Beekeepers

http://transitionwayland.org/beelieve