Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Generator: Add option to Skip columns/fields #435

Open
galexrt opened this issue Dec 26, 2024 · 2 comments · May be fixed by #436
Open

Generator: Add option to Skip columns/fields #435

galexrt opened this issue Dec 26, 2024 · 2 comments · May be fixed by #436
Labels
missing feature

Comments

@galexrt
Copy link

galexrt commented Dec 26, 2024

Is your feature request related to a problem? Please describe.

I'm adapting an existing database structure using the jet generator. It works fine but it generates the tables with all columns/fields.
I would like to be able to skip columns/fields from tables as well in the generator.
There doesn't seem to be a way to skip columns/fields using the TableSQLBuilder.UseColumn function like is possible with schemas/tables.

Describe the solution you'd like

Adding a Skip option to the TableSQLBuilderColumn structure (UseColumn(columnsFunc func(column metadata.Column) TableSQLBuilderColumn)) so columns/fields can be programmatically skipped.

Is this something that is already possible? If not, would this be something that could be implemented?
@galexrt galexrt linked a pull request Dec 26, 2024 that will close this issue
Open
@houten11
Copy link

Wondering what is the use case of those columns, since they will never be read or written.

@galexrt
Copy link
Author

galexrt commented Dec 29, 2024

The columns are used by other parts of an already existing application. The new application won't need these specific columns as they are only used by the existing application that hopefully will then soon be superseded by the new application.
So to add it would be nice to be able to hide/skip these columns so that they don't end up being used in the new application where they shouldn't be used as they won't be part of the table structure in the future. I hope this makes sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
missing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add generator skip option for table columns galexrt/jet
2 participants
@galexrt @houten11