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.

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

columns exclude configuration for list views #304

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gythaogg
Copy link
Collaborator

overriding ABSTRACT_ENTITY_COLUMNS_EXCLUDE for entities

This pull request includes changes to the apis_ontology module, focusing on the filter sets and forms to improve consistency and functionality. The most important changes include importing and utilizing new forms, modifying the Meta classes, and updating form classes to inherit from a new mixin.

Changes to filter sets:

  • Removed the import of GenericFilterSet from apis_ontology/filtersets.py.
  • Added the import of RelationSearchForm in apis_ontology/filtersets.py.
  • Updated the Meta class within TibScholRelationMixinFilterSet to use RelationSearchForm.
  • Modified the Meta classes for PlaceFilterSet, PersonFilterSet, WorkFilterSet, and InstanceFilterSet to inherit from their respective mixins. [1] [2] [3] [4]

Changes to forms:

  • Added imports for AbstractEntityFilterSetForm and GenericModelForm in apis_ontology/forms.py.
  • Introduced TibScholEntityMixinSearchForm with specified columns to exclude.
  • Updated PlaceSearchForm, PersonSearchForm, WorkSearchForm, and InstanceSearchForm to inherit from TibScholEntityMixinSearchForm. [1] [2] [3]
  • Added RelationSearchForm with specified columns to exclude.

@gythaogg gythaogg marked this pull request as draft February 19, 2025 08:10
overriding ABSTRACT_ENTITY_COLUMNS_EXCLUDE for entities
@gythaogg gythaogg force-pushed the remove-columns-from-listfilter branch from 1c7367c to 5f2cc66 Compare February 19, 2025 09:59
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