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

Feature/finalize wizard #64

Merged
merged 31 commits into from
Aug 13, 2020
Merged

Feature/finalize wizard #64

merged 31 commits into from
Aug 13, 2020

Conversation

otahontas
Copy link
Contributor

Description

This PR finalizes the wizard layout, adding proper functionality for creating a folder and adding objects inside it.

Related issues

Fixes #28 and #30

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Changes Made

  • Added folder support: Folders are posted to backend and added to redux store when clicking next in first wizard step. Folder-form is locked / disabled after creating folder.
  • Added support for adding objects to folders: when new xml / form based object is submitted succesfully to backend, it is also added to folder in backend and redux store
  • Added support for summary of metadata objects
  • Added support for object adding layout with accordion
  • Added possibility to clear forms
  • Added alert when user exists form
  • Added alert when user cancels wizard. If user wants to cancel, folder and all of its contents are deleted from backend
  • Couple of other bugfixes, see git log

Testing

  • Needs testing

Returning "fake" disable form here is a bit hacky solution, better one would be to disable Formik form.
Cancel now clears all wizard-related states to their initial status,
save+exit and publish are disabled when they should be disabled.
@otahontas otahontas self-assigned this Aug 13, 2020
@otahontas otahontas merged commit 4c407d5 into develop Aug 13, 2020
@otahontas otahontas deleted the feature/finalize-wizard branch August 13, 2020 13:48
@otahontas otahontas mentioned this pull request Aug 13, 2020
5 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.

2 participants