Maintenir la compatibilité Rails 6 pour la version "stable"

Soutien
1

Salut !

Les changements dans le commit FEATURE: Allow excluding groups from leaderboards (#37) · discourse/discourse-gamification@ac393c7 · GitHub, fichier db/migrate/20220623182333_add_excluded_groups_to_leaderboards.rb cassent lors de la mise à niveau sur une installation Discourse « stable ».

C’est déjà arrivé précédemment dans le plugin calendrier et a été corrigé par ce commit : FIX: Stable core still needs a 6.1 migration (#285) · discourse/discourse-calendar@259f7bf · GitHub

Pourriez-vous s’il vous plaît rétablir la compatibilité ?

Au fait, je n’ai pas vu de branche ou de tag pour les plugins. Ne serait-il pas agréable d’avoir une version « stable » des plugins comme pour Discourse ?

1 « J'aime »
2

Il existe .discourse-compatibility et lorsque cela n’est pas respecté, nous, chez Communiteq, conservons une grande quantité de forks fonctionnels pour la stabilité sur https://github.com/discourse-plugins/

3 « J'aime »
3

discourse-gamification est un plugin très récent, nous ne prévoyons pas de le prendre en charge dans la version stable de Discourse. La première version stable officiellement prise en charge sera dans la prochaine version de Discourse.

Cela dit, @Falco, j’ai utilisé ActiveRecord::Migration[6.0] à quelques endroits pour une compatibilité croisée facile.

1 « J'aime »
4

Sinon, je suppose que je peux donner une référence git sur mon fichier app.yaml pour épingler un commit précis du plugin, n’est-ce pas ?

5

Oui, toute commande ou option git normale fonctionnera là-bas.

2 « J'aime »
6

Je n’ai pas trouvé de commande git pratique pour faire cela, mais ce qui suit dans la configuration du conteneur a fait l’affaire :

hooks:
  after_code:
    - exec:
        cd: $home/plugins
        cmd:
          - git clone https://github.com/discourse/discourse-gamification && cd discourse-gamification && git checkout 48b2decfbefda6044953ca0413e45ca3cc18b156
1 « J'aime »
7

Ce commit ramène-t-il la compatibilité avec la version « stable » ?

1 « J'aime »
8

Je pense que oui, mais peut-être que @hyphalos peut confirmer que cela fonctionne pour lui ?

9

Je confirme : j’ai mis à jour vers le dernier commit tout en étant dans la branche « stable » de discourse et c’est bon maintenant

2 « J'aime »