Sync include columns with column name changes.#7617 #8348
Conversation
| @@ -60,6 +60,7 @@ export default class ColumnSchema extends BaseUISchema { | |||
| seqcycle: undefined, | |||
| colconstype: 'n', | |||
| genexpr: undefined, | |||
| isInclude: false | |||
There was a problem hiding this comment.
I don't think this is required. The problem is with syncing the column names. Normal columns works without any extra variables.
There was a problem hiding this comment.
The isInclude variable helps track which columns are added to the "Include Column" field in the create table modal, especially when column names change. This works similarly to the is_primary_key variable that tracks columns in the model's column field. This helps maintain proper tracking of column states. Let me know if you'd prefer a different approach!
There was a problem hiding this comment.
I don't see any is_foreign_key or is_unique_key variables. I don't think it is the right direction.
There was a problem hiding this comment.
Then what is the use of include_columns key?
Detail Info:- #7617