Rebuild-Black-Business / RBB-Website

Website to help connect black-owned businesses with consumers and resources
https://www.rebuildblackbusiness.com/
MIT License
119 stars 72 forks source link

Custom forms #297

Closed SeHarlan closed 4 years ago

SeHarlan commented 4 years ago

Describe your PR

Related to #295 Fixes #

Pages/Interfaces that will change

businesses.js - commented out Airtable embed added Custome form and added (closeOnOverlayClick={false}) to parent modal for better functionality (like a success message) AllyFeed.js - same as above except with AllySignUpForm

Screenshots / video of changes

Screen Shot 2020-06-30 at 11 32 57 AM Screen Shot 2020-06-30 at 11 32 46 AM

Steps to test

  1. Must add new .env keys (uses same values) GATSBY_AIRTABLE_API_KEY= GATSBY_AIRTABLE_BASE_ID=
  2. I tried a few different things here but Gatsby only seems to recognize keys on the FE when appended with GATSBY_

Additional notes

netlify[bot] commented 4 years ago

Deploy request for rebuild-black-business accepted.

Accepted with commit 7e04277f247dc63e9d4966b5084ff505a443709f

https://app.netlify.com/sites/rebuild-black-business/deploys/5eff6130ab6404000854f7ae

SeHarlan commented 4 years ago

Screen Shot 2020-07-01 at 12 05 40 PM

SeHarlan commented 4 years ago

the dynamic fetch wasn't catching the businesses "Other" category so I add it in conditionally on line 88 on AirtableServices.js

SeHarlan commented 4 years ago

Exports in components/index.js were left as airtable embeds cause I don't know everything those might be linked to and didn't wanna mess up any fallbacks that might exist.

magnificode commented 4 years ago

@SeHarlan This has been merged, can't thank you enough for the hard work here! This will help us to be able to move on some of our tasks to help improve the UX around business submissions!