Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Ithyx
Copy link
Member

@Ithyx Ithyx commented Sep 7, 2022

Description

Cette PR refactor les favoris et les votes pour les mettres au même niveau. Ceci permet de supprimer toutes les ressources nécessaires lorsqu'un utilisateur décide de supprimer une chanson.

🎉 Quality of life

La période des vote et la période d'ajout de chansons ont maintenant été fusionnées.

🐛 Bugfix

  • Fix d'un bug si le client envoyait une URL de chanson undefined.

🛠️ Refactoring

  • Le module de vote est maintenant un sous module du module de chansons
  • Le module de favoris est maintenant un sous module du module de chansons
  • Passage à Chelys 2.8.1

Checklist

  • Titre
  • Label
  • Catégorie

@Ithyx Ithyx added enhancement New feature or request WebSocket API Anything on the WebSocket API communications protocol Anything concerning the communications protocol labels Sep 7, 2022
@Ithyx Ithyx requested a review from hvdbm September 7, 2022 03:21
@Ithyx Ithyx self-assigned this Sep 7, 2022
hvdbm
hvdbm previously approved these changes Sep 8, 2022
Copy link
Member

@hvdbm hvdbm left a comment

Choose a reason for hiding this comment

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

🤞

Copy link
Member

@hvdbm hvdbm left a comment

Choose a reason for hiding this comment

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

@Ithyx Est-ce que tu pourrais faire la PR équivalente sur Yatga ? Autrement, on pourra pas utiliser Kalimba depuis un ancien client.

@Ithyx Ithyx merged commit 8670bd8 into main Sep 8, 2022
@Ithyx Ithyx deleted the feature/vote-rework branch September 8, 2022 19:44
@Ithyx Ithyx mentioned this pull request Sep 23, 2022
Ithyx added a commit that referenced this pull request Sep 23, 2022
* Migration vers Fly.io (#38)

* Added memory usage estimation to dashboard
* First dockerfile
* Added dockerfile and fly.io config
* Added deploy workflow and removed heroku procfile
* Added env file generation for workflow
* Finished CI for Fly.io deploy

* Refactorisation des votes pour permettre de voter pendant l'ajout de chansons (#40)

* TS maintenant target es6

* Refactor des votes et suppression des votes lorsqu'on supprime une chanson

* Suppression des favoris lors de la suppression d'une chanson

* Refactoring des favoris pour les hierarchiser sous le module des chansons

* Removed leftover log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

communications protocol Anything concerning the communications protocol enhancement New feature or request WebSocket API Anything on the WebSocket API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants