feat(snippets): filter by language #402
Conversation
wJoenn
force-pushed
the
feat/snippets/filter-by-language
branch
from
aaadfe5 to
437f48b
Compare
wJoenn
force-pushed
the
feat/snippets/filter-by-language
branch
from
437f48b to
5f43682
Compare
wJoenn
force-pushed
the
feat/snippets/filter-by-language
branch
from
5f43682 to
0232e6d
Compare
| @@ -8,6 +8,10 @@ def show | |||
| @snippet = Snippets::Builder.call(language: current_user.favourite_language) | |||
| @snippets = Snippet.includes(:user, :reactions).where(day: @day, challenge: @challenge).order(created_at: :desc) | |||
|
|
|||
| @language = params[:language] | |||
| @languages = @snippets.pluck(:language).uniq.sort | |||
There was a problem hiding this comment.
@snippets.distinct.order(:language).pluck(:language), tout dans la requête
There was a problem hiding this comment.
pluck c'est une requete SQL ?
Je pensais que c'etait juste du ruby du coup j'ai pas pensé a opti
There was a problem hiding this comment.
En fait le premier @snippets retourne un objet ActiveRecord, donc tu peux lui appliquer les méthodes associées. ActiveRecord emprunte pas mal de noms de méthode de Ruby, ça peut arriver de confondre
There was a problem hiding this comment.
irb(main):005> s.distinct.order(:language).pluck(:language)
/home/joenn/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rack-mini-profiler-3.1.1/lib/patches/db/pg.rb:51:in `prepare': PG::InvalidColumnReference: ERROR: for SELECT DISTINCT, ORDER BY expressions must appear in select list (ActiveRecord::StatementInvalid)
LINE 1: ...ay" = $1 AND "snippets"."challenge" = $2 ORDER BY "snippets"...
^There was a problem hiding this comment.
J'arrives pas a faire avec ActiveRecord et j'ai plus le temps de regarder sorry
|
@pil0u Tous les logos ont pas le meme aspect ratio donc on pourrait pas les avoir tous a la meme taille sans en deformer certains 🤔 |
|
C'est ce que je voulais dire par "pas la même taille", ça aurait été bien d'avoir que des images carrées. C'est bien avec la hauteur qu'il faut jouer oui 👍 |
Summary of changes and context
closes #264
None selected :
Go selected :
Mobile
Sanity checks