Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR: form-review:#2- added personal information component no css, pulled branch RB-design #15

Closed
wants to merge 51 commits into from

Conversation

nikitashrestha
Copy link
Contributor

No description provided.

src/components/media/Media.js Outdated Show resolved Hide resolved
src/components/preview/PersonalInfo.js Outdated Show resolved Hide resolved
src/pages/index.js Outdated Show resolved Hide resolved
src/components/preview/PersonalInfo.js Outdated Show resolved Hide resolved
src/components/preview/PersonalInfo.js Outdated Show resolved Hide resolved
src/components/preview/PersonalInfo.js Outdated Show resolved Hide resolved
@sagunji
Copy link
Contributor

sagunji commented Apr 17, 2020

@nikitashrestha Please add descriptions on your PR. you can bullet points things you have done. Its good that you mentioned pulled branch RB-design but it would be better to be added in the description and not in the title itself :D

Also, @Esha-Rajbhandari might push new changes because I have made some comments on it. Please, pull her branch once more as hers is merged or collaborate her to make changes on this branch.

src/assets/image.js Outdated Show resolved Hide resolved
const [preview, setPreview] = useState(false);

// Data source
const data = {
Copy link
Contributor

Choose a reason for hiding this comment

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

You can place it to /constant folder as mockData.js and import it in App file.

@nikitashrestha nikitashrestha deleted the form-preview branch May 13, 2020 08:06
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.

3 participants