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

Fix the Add.ts command || Add remove.ts || Add GetPlayList.ts || Make load.ts play the songs || Add StealPlaylist.ts #606

Merged
merged 18 commits into from
Jul 11, 2024

Conversation

Beelzebub2
Copy link
Contributor

trying to fix the command that wasn't working for me

Wasn't working for me, i will assume it doesn't work at all based on debugging 

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
this pull request stuff is confusing

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@Beelzebub2 Beelzebub2 mentioned this pull request Jul 11, 2024
Add the option to remove one song from the playlist

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
it is working for me at leas let me know if it doesn't 

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@Beelzebub2 Beelzebub2 changed the title Fix the Add.ts command Fix the Add.ts command and Add remove.ts Jul 11, 2024
(it's working for me)

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
make it work with getplaylist

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@Beelzebub2 Beelzebub2 changed the title Fix the Add.ts command and Add remove.ts Fix the Add.ts command and Add remove.ts and GetPlayList.ts Jul 11, 2024
@Beelzebub2
Copy link
Contributor Author

so for the error i was getting i fixed add
then i got annoyed at having to delete playlist to remove song so i added remove
and cause it was going well i added GetPlayList hope my spaghetti code doesn't ruin anything and actually works

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
upsie

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@Beelzebub2 Beelzebub2 changed the title Fix the Add.ts command and Add remove.ts and GetPlayList.ts Fix the Add.ts command || Add remove.ts || Add GetPlayList.ts || Make load.ts play the songs Jul 11, 2024
tiny upsie

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
same upsie

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
steals playlist of another user

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@Beelzebub2 Beelzebub2 changed the title Fix the Add.ts command || Add remove.ts || Add GetPlayList.ts || Make load.ts play the songs Fix the Add.ts command || Add remove.ts || Add GetPlayList.ts || Make load.ts play the songs || Add StealPlaylist.ts Jul 11, 2024
It can get another user's playlists or the user o called the command if no arg is given

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
And removed unnecessary console.log

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
wasn't working as intended now is

Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
Signed-off-by: Beelzebub <65181309+Beelzebub2@users.noreply.github.com>
@LucasB25
Copy link
Collaborator

It’s okay, can I test your code?
regarding your last pull requests I saw that you removed it?

@Beelzebub2
Copy link
Contributor Author

about these commands i think you can test everything

@LucasB25 LucasB25 merged commit 3d31657 into appujet:main Jul 11, 2024
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.

2 participants