Skip to content

Agregar Name a Degree#985

Merged
RenzoMinelli merged 5 commits intomasterfrom
rm/add-degree-title
Feb 5, 2026
Merged

Agregar Name a Degree#985
RenzoMinelli merged 5 commits intomasterfrom
rm/add-degree-title

Conversation

@RenzoMinelli
Copy link
Copy Markdown
Collaborator

Agregar title a degrees para tener dropdown con valor más legible.

@RenzoMinelli RenzoMinelli changed the title Rm/add degree title Agregar title a Degree Jan 29, 2026
Comment thread app/views/users/degrees/edit.html.erb Outdated
Comment thread db/schema.rb Outdated
Comment thread config/degrees.yml Outdated
shared:
- bedelias_name: "INGENIERÍA EN COMPUTACIÓN"
id: "computacion"
title: "Ingeniería en Computación"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Por qué no usar el bedelías_name que ya tenemos?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Siento que pueden ser dos cosas diferentes, por un lado, cómo aparece en Bedelías y, por otro, el nombre de la carrera. Es posible que en Bedelías figure con alguna falta ortográfica o abreviación.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Si yo opino lo mismo, me parece que tener un campo en la tabla degree nos va a dar mas flexibilidad si es que de repente cambian el formato en bedelias y queremos mantener otro formato nosotros.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Entiendo lo que dicen y coincido. Mi propuesta era usar bedelías_name para llenar el campo de title

Comment thread .gitignore Outdated
RenzoMinelli and others added 3 commits February 5, 2026 10:22
- Add migration to add title column to degrees table
- Add title field to degrees.yml configuration
- Update YmlLoader to upsert title when loading degrees
- Update cambiar carrera dropdown to show degree titles instead of humanized IDs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add title to degrees fixture and factory
- Add title assertions to yml_loader_spec
- Update changing_degrees_spec to use titles instead of humanized IDs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@RenzoMinelli RenzoMinelli changed the title Agregar title a Degree Agregar Name a Degree Feb 5, 2026
@RenzoMinelli RenzoMinelli merged commit 046df5f into master Feb 5, 2026
5 checks passed
@RenzoMinelli RenzoMinelli deleted the rm/add-degree-title branch February 5, 2026 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants