Consolidate app/queries/ and app/queriers/ into a single query-objects folder
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
Comienza leyendo Admin::WorkshopConcerns y el punto de entrada Admin::WorkshopsController#show; después, localiza AdminWorkshopAttendeeFlags en app/queriers/. Muévelo a app/queries/ sin cambiar sus llamadores y ejecuta las specs de Admin::WorkshopsController#show y del dashboard. Está hecho cuando app/queriers/ haya desaparecido, la clase se resuelva desde app/queries/ y ambas áreas de pruebas indicadas pasen.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
AdminWorkshopAttendeeFlagsfromapp/queriers/toapp/queries/and deleteapp/queriers/. - Leave the three existing call shapes alone for now — normalising
DashboardQuery/SponsorsSearchinterfaces is out of scope and needs its own discussion.
Acceptance
app/queriers/no longer exists;AdminWorkshopAttendeeFlagsresolves fromapp/queries/.- Callers unchanged (
Admin::WorkshopConcernsreferences the same class name). Admin::WorkshopsController#showand the dashboard specs pass.
- Lenguaje dominante
- Ruby
- Estrellas
- 104
- Forks
- 205
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 81
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de codebar/planner
-
performance
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
good first issue tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
good first issue tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Chapter show pages allocate ~21k+ objects per render for large chapters (18% of app allocations)Abiertoperformance
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Los mantenedores suelen responder en 1 día
Todos los issues de codebar/planner
Issues similares
-
Obsolete link to RubyMine docsAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
rubocop/ruby-style-guide#984 ·
-
Dev Portal: Wizard refereshAbiertointernal product:dev-portal release-docs
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Kong/developer.konghq.com#7393 ·
Los mantenedores suelen responder en 1 día
-
Unused stubs and fixturesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
fastlane/fastlane#30286 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
TheOdinProject/curriculum#31432 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area/web interface
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día