Skip to content

Feature/129 character name class editing#150

Merged
aphexlog merged 4 commits intomainfrom
feature/129-character-name-class-editing
Jan 16, 2026
Merged

Feature/129 character name class editing#150
aphexlog merged 4 commits intomainfrom
feature/129-character-name-class-editing

Conversation

@aphexlog
Copy link
Contributor

No description provided.

Aaron West added 4 commits January 16, 2026 11:01
- Create CharacterCreation component with name, race, and class selection
- Show modal when starting new Game Master conversation without existing character
- Allow users to customize character before beginning adventure
- Add validation for character name (required, max 50 chars)
- Support cancel action with default character creation
- Integrate createCharacter function to save to database
- Default stats: STR(10), DEX(12), CON(14), INT(16), WIS(13), CHA(11)

Addresses #129
- Add owner field to GameMasterCharacter schema
- Update authorization to allow authenticated read access
- Implement client-side filtering workaround for Amplify GraphQL bug
- Fetch all characters and filter by conversationId in client code
- Add retry mechanism with 1-second delays (up to 3 attempts)
- Add comprehensive logging for debugging character fetch/create
- Increase propagation delay to 2 seconds after character creation
- Character now persists correctly across page refreshes

This fixes the issue where characters were being created but not found
on refresh due to Amplify's server-side filter authorization bug.
- Add isLoadingCharacter state to track loading status
- Show loading skeleton in GameMasterHud while character is fetching
- Hide character displays until data is loaded (no more placeholder flash)
- Remove fallback values that caused flashing default data
- Pass loading state to both desktop and mobile character displays
- Set loading state at start and end of fetchCharacter
- Mobile character sheet only shows when character data exists

Fixes visual glitch where 'Adventurer' and default stats appeared
briefly before real character data loaded from database.
- Remove double-click delete (conflicted with rename)
- Add pencil icon button for rename (hover to show)
- Single-click delete with confirmation dialog
- Remove deleteHotId state (no longer needed)
- Remove double-click rename functionality
- Cleaner, more intuitive interaction model

Now users can:
- Click conversation to open
- Hover → pencil icon to rename
- Hover → trash icon to delete (with confirmation)
- Use 'Select' mode for bulk delete

No more conflicting double-click actions!
@aphexlog aphexlog merged commit fe8302d into main Jan 16, 2026
1 check passed
@aphexlog aphexlog deleted the feature/129-character-name-class-editing branch January 16, 2026 18:39
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.

1 participant