Skip to content

simpleInput pasted to data.js and import to editProject.js and addProject.js#1490

Merged
StephenTheDev1001 merged 2 commits intohackforla:developmentfrom
StephenTheDev1001:simpleInput-to-own-obj-1483
Aug 29, 2023
Merged

simpleInput pasted to data.js and import to editProject.js and addProject.js#1490
StephenTheDev1001 merged 2 commits intohackforla:developmentfrom
StephenTheDev1001:simpleInput-to-own-obj-1483

Conversation

@StephenTheDev1001
Copy link
Contributor

Fixes #1483

What changes did you make and why did you make them ?

  • refactored to avoid repeating simpleInputs in multiple files
  • created client/src/components/data.js file and pasted simpleInputs array
  • refactor editProject.js to import simpleInputs from created file
  • refactor addProject.js to import simpleInputs from created file

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

no visual changes

@StephenTheDev1001 StephenTheDev1001 changed the title simpleInput pasted to data.js and import simpleInput to editProject.js and addProject.js simpleInput pasted to data.js and import to editProject.js and addProject.js Aug 29, 2023
@StephenTheDev1001 StephenTheDev1001 force-pushed the simpleInput-to-own-obj-1483 branch from 177a83d to 471f1f7 Compare August 29, 2023 04:56
Copy link
Member

@trillium trillium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does exactly the job we needed, Thanks @StephenTheDev1001 !

@StephenTheDev1001 StephenTheDev1001 merged commit e1494f3 into hackforla:development Aug 29, 2023
@StephenTheDev1001 StephenTheDev1001 deleted the simpleInput-to-own-obj-1483 branch August 29, 2023 17:29
@trillium
Copy link
Member

Yeah!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move simpleInput to it's own data.js object

2 participants