Conversation
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
…ialog AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
…tion changes AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
…nt, style limit) AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
- Add ROW_FORMATS constant to AbstractContext (empty), MySQLContext and MariaDBContext - Add row_format field to MySQLTable/MariaDBTable; fetched from information_schema - Add convert_data transient flag (compare=False) to control CONVERT TO CHARACTER SET - Fix MySQLTable.alter_collation() signature to accept collation_name (was buggy) - Add alter_row_format() to MySQLTable and MariaDBTable - Wire convert_data and row_format into EditTableModel and MainFrameController - Fix server_version stored as instance attribute in MySQL/MariaDB after_connect AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
Clarify the distinct roles of _parse_type (DDL column string parsing), _get_field_type_name (pymysql type code resolution), and get_result_column_datatypes (query result metadata mapping) in both MySQL and MariaDB contexts. AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
Includes pending UI changes (views, fbp layout, database_options controller, settings, dataview, PostgreSQL/SQLite context updates). AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
…d string literals AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
- Implement on_apply_record and on_cancel_record in MainFrameController (previously stubs in the base view); changes pending in NEW_RECORDS are now saved on Apply and discarded on Cancel - Add do_apply_records and do_cancel_records to TableRecordsController - Fix UnboundLocalError in _count_table_records: total_rows was left unassigned when dict(row) raised an exception - Wrap _load_records_page and _update_records_label calls inside _on_records_count_complete with try/except to prevent unhandled exceptions crashing the app via wx.CallAfter AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
- Remove broken link to ROADMAP.md (merged into PROJECT_STATUS in March) - Refresh README "Recent updates" with latest features - Bump PROJECT_STATUS last-updated date to 2026-04-27 - Add 7 entries to "Recently Added": autocomplete, table execution flow, row_format/convert_data, windows/main refactoring, ColumnContentDialog, database action buttons, tree explorer state preservation AI-Assisted-By: Cline AI-Contribution: 80% Tracked-By: CodeShield AI
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This merge request consolidates several improvements across i18n, query execution, dataview handling, engine contexts, and the main UI workflow.
Main changes
skip_before_connect/skip_after_connectbehavior in engine context connectionsrow_formatandconvert_dataWhy
These changes improve consistency in translation formatting, make query execution more robust and user-friendly, simplify parts of the UI/data model architecture, and strengthen engine-specific behavior and typing.
Notes for reviewers