Skip to content

Conversation

@subdavis
Copy link
Contributor

@subdavis subdavis commented May 4, 2022

Adds proper KPF goroup support.

part of #850

@subdavis subdavis changed the base branch from main to 1.9 May 18, 2022 20:14
@subdavis subdavis force-pushed the server/kpf-support-v2 branch from 21bd715 to d5d3ec2 Compare May 19, 2022 16:44
@subdavis subdavis marked this pull request as ready for review May 19, 2022 17:22
@subdavis subdavis requested a review from BryonLewis May 19, 2022 17:22
Copy link
Collaborator

@BryonLewis BryonLewis left a comment

Choose a reason for hiding this comment

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

The uploading a new dataset with the combine YML file or importing it directly seems to be fine on web.

There is a decent delay when importing combined ymls as compared to track.json/CSV files, but I'm not too worried about it because it is a feature that won't be used often.

There is an issue with client not properly deleting the groups when importing and the subsequent reloading, but I'll log that in the 1.9 review.

Just noticed some minor things, probably want to add version to the exported track json files.

@subdavis
Copy link
Contributor Author

Updated. I untangled the mess of _get_data_by_type and I think it's a lot better now. Type safety continues to improve :)

Copy link
Collaborator

@BryonLewis BryonLewis left a comment

Choose a reason for hiding this comment

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

looks good.

@subdavis subdavis merged commit ae19116 into 1.9 May 20, 2022
@subdavis subdavis deleted the server/kpf-support-v2 branch May 20, 2022 19:49
subdavis added a commit that referenced this pull request May 25, 2022
* Activity Group Support (#1205)

* Activity Groups

* Move Image Enhancement

* wip

* GroupItem

* Refactor Filter Controls

* Complete backend support for activity groups with migrations

* Refactor of GroupStore, Filter Controls, Filter List

* Group filtering support

* test fixes

* sidebar

* sidebar update

* Respond to comments

* Fix tests

* fixing final test (#1234)

Co-authored-by: BryonLewis <61746913+BryonLewis@users.noreply.github.com>

* Release 1.9.0-beta.1

* Group Creation (#1237)

* WIP Group Creation

* Edit type and select.  Refactor TypePicker

* Progress toward creation and editing of activity groups

* Group membership add and remove

* Update package.json

* Complete Activity Group Support (#1238)

* Progress on activity group editing

* Progress on correct editing behavior

* Finalize editing

* Documentation updates

* Update documentation

* Add shortcut docs

* Wip

* KPF import support desktop

* icon

* Fix docs add tests

* whatdo

* Remove multi-file inputs

* Upgrade node for electron-builder

* Respond to comments

* Backend fixes

* Add readOnlyMode to TypePicker in GroupList

* Update package.json

* Include buildResources

* Server/kpf support v2 (#1235)

* Work in progress

* Finalize KPF support

* Fully move to new annotation schema in all parsers

* Liint fix

* Add extra meta in zip extraction

* Fix

* Fix merge conflict

Co-authored-by: BryonLewis <61746913+BryonLewis@users.noreply.github.com>
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.

2 participants