Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

Consolidate app/queries/ and app/queriers/ into a single query-objects folder

Fechada Para iniciantes
  • Algo errado?
#2,943 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
84/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
rails, ruby
Domínio
backend

Direção de pesquisa

Comece lendo Admin::WorkshopConcerns e o ponto de entrada Admin::WorkshopsController#show; em seguida, localize AdminWorkshopAttendeeFlags em app/queriers/. Mova-o para app/queries/ sem alterar seus chamadores e execute as specs de Admin::WorkshopsController#show e do dashboard. Está concluído quando app/queriers/ não existir mais, a classe for resolvida a partir de app/queries/ e ambas as áreas de teste listadas passarem.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

good first issue refactoring tech debt

Problem

The repo has two folders for the same concept, with three different class shapes between them:

Folder Class Shape Used by
app/queries/ DashboardQuery class-level methods (upcoming_events, total_upcoming_events_count) DashboardController
app/queries/ SponsorsSearch ActiveModel::Model object with attr_accessor :name, :chapter Admin::SponsorsController
app/queriers/ AdminWorkshopAttendeeFlags self.for_members(member_ids) → instance #to_h Admin::WorkshopConcerns

queriers is not a name Rails developers recognise as an autoload convention (app/queries is), and the split isn't semantic — AdminWorkshopAttendeeFlags is the same kind of code as DashboardQuery: batched aggregate queries for controllers.

Proposed change

  • Move AdminWorkshopAttendeeFlags from app/queriers/ to app/queries/ and delete app/queriers/.
  • Leave the three existing call shapes alone for now — normalising DashboardQuery/SponsorsSearch interfaces is out of scope and needs its own discussion.

Acceptance

  • app/queriers/ no longer exists; AdminWorkshopAttendeeFlags resolves from app/queries/.
  • Callers unchanged (Admin::WorkshopConcerns references the same class name).
  • Admin::WorkshopsController#show and the dashboard specs pass.
Linguagem predominante
Ruby
Estrelas
104
Forks
205
Merge médio
1d 2h
PRs com merge (30d)
81

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de codebar/planner

Todas as issues de codebar/planner

Issues semelhantes

Mais issues de Ruby

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.