AritDeveloperCircle / findyourpeople

MIT License
8 stars 0 forks source link

Neka manager form style, functionality #134

Closed nekablair closed 1 year ago

nekablair commented 1 year ago

What type of PR is this? (check all applicable)

Description of what this Pull Request does

style rest of manager form will still need to add tailwind css

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Tailwind, still being added

https://github.com/AritDeveloperCircle/findyourpeople/assets/57186734/c781b6e2-2269-4b84-bb91-8a1b36d2d814

Added/updated tests?

TaylorDJones11 commented 1 year ago

Looks good! @istoga My only suggestion would be about the vision/description fields instead of inputs making them textarea. The input looks like the user may be limited to characters, whereas a textarea shows the user there is more space allocated.

Screenshot 2023-06-08 at 11 30 32
nekablair commented 1 year ago

@TaylorDJones11 that's a great point, I will change that

nekablair commented 1 year ago

I cannot check firebase, states quota exceeded

Domicaren1 commented 1 year ago

@istoga have you started working on the styling? If no, I can take it up.

nekablair commented 1 year ago

@Domicaren1 I started, but please feel free to continue, we want to get everything finished on time. Thanks for offering!

nekablair commented 1 year ago

Thanks @ibimina for your help, greatly appreciated.

Domicaren1 commented 1 year ago

Ok @istoga . I'll start. Thanks

msarit commented 1 year ago

@Domicaren1 @istoga Please let me know when this PR is ready for review 🙏🏾

nekablair commented 1 year ago

@msarit I finished the functionality and cleaned up the code for creation, but in order to do editing, it looks like that should happen over on the manager dashboard with a button. I will head over there and place a button on the dashboard. And will work on the editing portion. If there any suggestions, or I am not heading in the right way, please, anyone let me know, thanks!

nekablair commented 1 year ago

@msarit @jjleigh can we merge?

nekablair commented 1 year ago

@msarit I added a simple alert box so users would know that their community was added to the database. I also checked and it looks like the one I put in actually did update to the database. I entered one called Yellowstone National Park.

netlify[bot] commented 1 year ago

Deploy Preview for effortless-dragon-109f52 failed.

Name Link
Latest commit b77b9b8229fbc32870a8be54cb9055c3e1801cff
Latest deploy log https://app.netlify.com/sites/effortless-dragon-109f52/deploys/649a1dd40f2471000896d2d9