Skip to content

Feat/delete entity#1151

Open
andrei-antal wants to merge 1 commit intomasterfrom
feat/delete-entity
Open

Feat/delete entity#1151
andrei-antal wants to merge 1 commit intomasterfrom
feat/delete-entity

Conversation

@andrei-antal
Copy link
Contributor

Delete entity functionality (partial)

  • added actions for entities that can be defined in the yaml under the action property of the EntityList block and will render in the interface as buttons
  • frontend: refactor Entity and EntityList - now the whole entity is passed down from the list, and data decomposition is handled in the entity component. When actions are triggered, the whole entity is passed up to the parent
  • added the DeleteEntity block
  • integrated the delete flow in the product-list pipeline
  • leftovers: removed the crud reference pipelines

NOTE: ATM delete does not refresh the entity list on the fronted, although the delete is indeed executed. So whenever a delete action is emitted we need to manually refresh in order to see the results

@CarmenPopoviciu
Copy link
Contributor

@andrei-antal can you pls squash these commits into one, if possible. It would make my rebase attempt much easier <3

feat(entity-delete): update list pipeline for actions and pass the action to the delete blocks

feat(entity-delete): Remove stand alone reference pipelines

feat(entity-delete): Refactor frontend code: pass the entity to the EntityComponent and hadle decomposition in that component

feat(entity-delete): Refactor pipe code, comment flatMap section as it blocks the create flow

feat(entity-delete): update formatting
@teropa
Copy link
Contributor

teropa commented Apr 26, 2018

Cool! The storage blocks have been switched from in-memory to postgres just now. Perhaps good to have a word with @leshy on getting the delete operation covered as well

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.

3 participants