Skip to content

Fix with column name - #41

Draft
podhmo wants to merge 2 commits into
masterfrom
fix-with-column-name
Draft

Fix with column name#41
podhmo wants to merge 2 commits into
masterfrom
fix-with-column-name

Conversation

@podhmo

@podhmo podhmo commented Sep 20, 2023

Copy link
Copy Markdown
Owner

@podhmo

podhmo commented Sep 20, 2023

Copy link
Copy Markdown
Owner Author

This line is also column_name is not supported?

for c in prop.local_columns:
yield FOREIGNKEY, walker.mapper._props[c.name], {

)
history.pop()
elif action == FOREIGNKEY: # ColumnProperty
sub = {}

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I'm not confident

@podhmo

podhmo commented Sep 26, 2023

Copy link
Copy Markdown
Owner Author

sorry, changes to low priority for my daily life

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