swciitg / PortfolioBuilder

5 stars 39 forks source link

Add the option to delete any section of the portfolio #10

Closed deepak-2605 closed 11 months ago

deepak-2605 commented 11 months ago

Add the option to delete any section of the portfolio. For example, if a user doesn't have any experience, they may not want to include an experience section in their portfolio. Provide the option to remove a particular section. The code and preview should get updated accordingly. This option should be provided in front of every section. When selecting that option, that field in form should be disabled. Also, provide an option for the user to add that section again. When selecting the option to add a section, that field should be enabled again, and the code and preview parts should also be updated accordingly.

NOTE: Do write comments in your code and write the code in separate files and import those files in the main page. Give meaningful names to files and variables . Kindly avoid the authentication for now. Directly go to http://localhost:3000/portfolio

Tushar98644 commented 11 months ago

Kindly assign this issue to me

deepak-2605 commented 11 months ago

@Tushar98644 Assigned to you

Dhruv127 commented 11 months ago

please see PR #13 , do tell if need to add option to delete more sections also

deepak-2605 commented 11 months ago

@Dhruv127, we appreciate your interest in contributing. However, please be informed that this particular issue has already been assigned to @Tushar98644. Therefore, we are unable to consider your pull request for this issue at this time. We encourage you to explore our other issues. Thank you for your understanding and willingness to contribute!

deepak-2605 commented 11 months ago

@Tushar98644 Try to complete this by Thursday EOD.

Ayush-Singh77 commented 11 months ago

hey could you please assign me this task as i also want to contribute to it.

prakharbhardwaj1504 commented 11 months ago

I will deliver it up to your expectation, kindly assign...

Dhruv127 commented 11 months ago

Can you merge PR#13 now ? Bro i have also given some time on it, do tell if i need to improve something on it.. will do it.

AnkitGurwan commented 11 months ago

@Tushar98644 we have de-assigned you.

@Dhruv127 we are reviewing your PR now.

Dhruv127 commented 11 months ago

please see PR #23