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

feat: advanced view of asset index #1384

Open
wants to merge 94 commits into
base: main
Choose a base branch
from

Conversation

DonKoko
Copy link
Contributor

@DonKoko DonKoko commented Oct 31, 2024

Resolves #1322

DonKoko and others added 30 commits September 19, 2024 18:16
- created basic layout of index pagination
- Added AssetIndexSettings model
- Created migration to populate the model for each UserOrganization
- adding columns Json field in AssetIndexSettings
- Setting deafualt value for it which consists of the default fields every user has.
- creating a migration that updates all the entries in AssetIndexSettings by adding the values of the custom fields in the related organization
- Building basic ui with checkboxes for the active columns
- adding real relationship between settings and org
- handling updating of settings when updating custom fields
- creating settings for personal org of new users
- creating settings when user creates a new workspace/org
- separated data fetching for assets into 2 different functions for each mode
- added filter and sort buttons with popovers
- added filters json field in AssetIndexSettings
- changed mode swapping to not use optimistic approach, because we need to actually reload the assets page with the correct mode
- implemented a 'changing mode' loader screen
- made an kind of acceptable version of the sticky header. can still be improved a lot
…ion fields and custom fields with type single and multi line
…stom fields are created. Also implemented an extra validation that runs when settings are loaded and makes sure the data in the AssetIndexSettings.columns is valid
@DonKoko DonKoko linked an issue Oct 31, 2024 that may be closed by this pull request
100 tasks
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.

feat: advanced view of asset index
2 participants