Skip to content

feat(edition): allow send config option / url call for dom / add, modify, delete disabled button config#1048

Merged
PhilippeLafreniere18 merged 5 commits into
nextfrom
editionFeat
Jun 13, 2022
Merged

feat(edition): allow send config option / url call for dom / add, modify, delete disabled button config#1048
PhilippeLafreniere18 merged 5 commits into
nextfrom
editionFeat

Conversation

@PhilippeLafreniere18

@PhilippeLafreniere18 PhilippeLafreniere18 commented Apr 27, 2022

Copy link
Copy Markdown
Contributor

Please check if the PR fulfills these requirements

What is the current behavior? (You can also link to an open issue here)
Nothing discriminate which source field you really want to send on add/modify action
Predefined url are not considered for relation http call
add, remove and modify buttons are always available
Edit mode shows all other features (in edit mode or not) and causes some problems with multiple domain value

What is the new behavior?
"send" config allow to send or not send sourceFields on add/modify action
"url" can now be part of relation options config
add, remove and modify buttons can now be set to disabled directly by layer edition options
Edit mode only shows the feature in edit mode

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

@PhilippeLafreniere18 PhilippeLafreniere18 changed the title feat(edition): allow send config option and url call for dom feat(edition): allow send config option / url call for dom / add, modify, delete disabled button config May 12, 2022
@PhilippeLafreniere18 PhilippeLafreniere18 merged commit 31c57f8 into next Jun 13, 2022
@PhilippeLafreniere18 PhilippeLafreniere18 deleted the editionFeat branch September 23, 2022 16:24
cbourget pushed a commit that referenced this pull request Mar 21, 2023
…ify, delete disabled button config (#1048)

* feat(edition): allow send config option and url call for dom

* feat(edition): add button disabled config

* refactor(edition): Edit mode only with state view

* feat(edition): force column value

* fix(edition): fix edition mode dom value set
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.

1 participant