pycon icon indicating copy to clipboard operation
pycon copied to clipboard

Change conference name from a multi-lingual field to a CharField

Open marcoacierno opened this issue 1 year ago • 2 comments

Currently the conference name is a multi-lingual field but there is no need for it. We should change it to use a simple CharField

marcoacierno avatar Sep 24 '24 13:09 marcoacierno

@claude work on this issue according to the issue description

marcoacierno avatar Aug 03 '25 20:08 marcoacierno

Claude finished @marcoacierno's task —— View jobclaude/issue-4066-20250803-2032Create PR ➔


Issue #4066: Change conference name from multi-lingual field to CharField ✅

Todo List:

  • [x] Read CLAUDE.md for repo setup instructions (no CLAUDE.md found)
  • [x] Find conference model definitions in the codebase
  • [x] Identify current multi-lingual field implementation
  • [x] Change field to CharField
  • [x] Generate database migration
  • [x] Update GraphQL type resolver
  • [x] Commit and push changes

Changes Made:

  1. Model Update: Changed Conference.name from I18nCharField to models.CharField in backend/conferences/models/conference.py:27
  2. Import Cleanup: Removed unused I18nCharField import from the model file
  3. Migration: Created migration 0055_change_conference_name_to_charfield.py to handle the database schema change
  4. GraphQL Fix: Updated Conference.name field in GraphQL types to use regular field instead of localized resolver

What Was Done:

  • ✅ Converted the multi-lingual conference name field to a simple CharField as requested
  • ✅ Created proper Django migration to handle the database schema change
  • ✅ Updated GraphQL API to work with the non-localized field
  • ✅ Cleaned up unused imports

Note:

Since I don't have access to run tests directly, please ensure to run the test suite and any linting tools (like ruff) after creating the PR to verify everything works correctly.


claude[bot] avatar Aug 03 '25 20:08 claude[bot]