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

Metadata testing #14

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Metadata testing #14

wants to merge 15 commits into from

Conversation

mororor
Copy link
Collaborator

@mororor mororor commented Apr 15, 2019

MetadataEditor and MetadataHub:
Basic functionality is complete. Code is in orderly state.

still very buggy. I.e. UI does not work
- controller has most of the basic functionality
- store works as expected so far
added aingleType property to Schema,
added support for single Type property to Controller.
UI: Basic editing features
- ./store.js: useMetadata() does not work as expected
- cleanup and fruther functionality required
small schema improvements
added state to metadata input component
metadata object now owns an values object like:
values: { value: { status: 'actual', value: 'value'}
where value key equal value string value.
status can be:
'actual': as in tripleStore
'delete': delete from tripleStore
'draft': add to tripleStore
also added ui option to delete metadata
- started generalization to allow for linking to resources in MetadataEditor
- implemented this feature draft-like
code clean up
some ui eye-candy
some ui key-events
- everything really sketchy and buggy

for the purpose some file renaming and code movement.
extracted some stuff to ui
@mororor mororor requested a review from Frando April 15, 2019 10:38
@Frando
Copy link
Member

Frando commented Apr 15, 2019

Very nice, will properly review tomorrow.

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.

None yet

2 participants