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

Сокращения #18

Open
MarshalX opened this issue Aug 25, 2019 · 13 comments
Open

Сокращения #18

MarshalX opened this issue Aug 25, 2019 · 13 comments
Labels
feature Новая функция или улучшение старой help wanted Требуется дополнительное внимание

Comments

@MarshalX
Copy link
Owner

Всякие .like() и .dislike(), .add_to_playlist() у треков, альмобов, плейлистов и т.д. и т.п.

Необходимо сформировать список таких шорткатов.

@MarshalX MarshalX added feature Новая функция или улучшение старой help wanted Требуется дополнительное внимание labels Aug 25, 2019
@zhanymkanov
Copy link
Contributor

Привет! Я правильно понял, что к каждому из требуемых классов нужно добавить вышеперечисленные методы, выполняющие соответствующий функционал посредством вызова, например, _like_action?

@MarshalX
Copy link
Owner Author

MarshalX commented Oct 5, 2019

@n1EzeR Привет, да, верно. И передавать сразу в эти методы необходимые данные из текущего класса. Например что бы поставить лайк на тоек нужно передать его ID и т.д.

@zhanymkanov
Copy link
Contributor

Окей, спасибо за быстрый ответ. Возьмусь.

MarshalX pushed a commit that referenced this issue Oct 6, 2019
…ylist #18

* Добавлено значение remove=False во время вызова _like_action из вызывающих функций

* Добавлено значение remove=False во время вызова _dislike_action из вызывающих функций

* Добавил like/dislike функционал

* Использовать self.track_id, вместо self.id
@MarshalX
Copy link
Owner Author

MarshalX commented Oct 6, 2019

Супер, большое спасибо! Начало положено! Ведь это далеко не все сокращения что можно придумать. Всякие удаление/добавления трека в плейлист как метод у трека и прочее-прочее.

@zhanymkanov
Copy link
Contributor

Если никто не займется, то добавление/удаление в плейлист попробую на следующих выходных. Были (и есть) некоторые проблемки с revision'ами, решил пока так залить.

@MarshalX
Copy link
Owner Author

MarshalX commented Oct 6, 2019

Вообще было бы классно тут составить список сокращений

@MarshalX
Copy link
Owner Author

При каждом выводе треков надо конкатенировать имена артистов, ибо их может быть несколько. Было бы классно это вынести в метод либы, в класс Track

@MarshalX
Copy link
Owner Author

Схороню тут. У класса MixLink не хватает метода для загрузки cover'a.

@MarshalX
Copy link
Owner Author

Загрузка рекламного сообщения у Promotion

@lemonpaul
Copy link
Contributor

При каждом выводе треков надо конкатенировать имена артистов, ибо их может быть несколько. Было бы классно это вынести в метод либы, в класс Track

Имена исполнителей для трека следует получать как-то так:

f'{", ".join([artist.name for artist in track.artists])}'

Такой же метод нужен и для класса Album.

@MarshalX
Copy link
Owner Author

MarshalX commented Feb 1, 2021

При каждом выводе треков надо конкатенировать имена артистов, ибо их может быть несколько. Было бы классно это вынести в метод либы, в класс Track

Имена исполнителей для трека следует получать как-то так:

f'{", ".join([artist.name for artist in track.artists])}'

Такой же метод нужен и для класса Album.

@glebliutsko даже в примерах я такое часто использовал. Не хотим как проперти? Тут в целом каким-то мысли лежат в сообщениях.

Тоже самое с получением полных версий треков. Вот это самое client.tracks([track_short.track_id for track_short in self.tracks]) (код писал из головы чтобы донести суть). Я тоже очень часто такое писал при разработке бота

@alexbelan1999
Copy link

@MarshalX необходимо добавить в класс Track метод, который бы возвращал строку с именами исполнителей трека?

@MarshalX
Copy link
Owner Author

@MarshalX необходимо добавить в класс Track метод, который бы возвращал строку с именами исполнителей трека?

Можно. Не факт что только в Track. Может быть есть еще другие классы

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Новая функция или улучшение старой help wanted Требуется дополнительное внимание
Projects
None yet
Development

No branches or pull requests

4 participants