PlaylistAdda icon indicating copy to clipboard operation
PlaylistAdda copied to clipboard

Add JSON file for adding playlists.

Open utkarsh1311 opened this issue 2 years ago • 4 comments

Reason/Context

Please try answering few of those questions

  • Why we need this improvement?

Currently, each playlist is added manually by users by directly editing the index.html file. This can cause bugs as they can miss certain attributes for the html tags used to render the playlist. Every playlist card has the same UI. The only things that are changing is the data specific to the user.

  • How will this change help?

If we have a json file in which users can add their personal data, then it will greatly reduce the chances for mistakes in the index.html file. It will also be easier to update and maintain user data. They can easily change their data without affecting the original webpage.

  • What is the motivation?

Currently the index.html file has a lot of duplicated code used to display the various playlists. This can be reduced by adding a json file to manage user data.

Description

Please try answering few of those questions

  • What changes have to be introduced? Addition of a JSON file to keep important info about the users. Changes to the index.js file to create the playlist card dynamically and add it to the index.html file.
  • Will this be a breaking change? No
  • How could it be implemented/designed? Add a JSON file and use document.innerHTML properties to dynamically generate playlist cards.

I would like to work on this if this issue is accepted. Please assign it to me.

utkarsh1311 avatar Oct 06 '22 15:10 utkarsh1311

@utkarsh1311 go ahead

vinayakgavariya avatar Oct 07 '22 03:10 vinayakgavariya

Working on it.

utkarsh1311 avatar Oct 07 '22 05:10 utkarsh1311

Hi @utkarsh1311 Are you still working in this enhancement?

futoid avatar Oct 12 '22 13:10 futoid

Hey @futoid. Yes, I am working on this.

utkarsh1311 avatar Oct 12 '22 19:10 utkarsh1311