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

🐛 fix(range): ajout de la possibilité de changer la value en JS [DS-3853] #1025

Merged
merged 3 commits into from
Oct 28, 2024

Conversation

keryanS
Copy link
Contributor

@keryanS keryanS commented Oct 14, 2024

  • Le composant est mis à jour graphiquement au changement de value des inputs en js.

@keryanS keryanS added correctif Quelque chose ne fonctionne pas à valider maj 1.12.1 labels Oct 14, 2024
@keryanS keryanS added this to the 1.13.0 milestone Oct 14, 2024
@keryanS keryanS requested a review from lab9fr October 14, 2024 12:49
@keryanS keryanS self-assigned this Oct 14, 2024
@keryanS keryanS changed the base branch from main to dev-1.13.0 October 22, 2024 10:34
@lab9fr lab9fr changed the title 🐛 fix(range): ajout de la possibilité en changer la value en JS [DS-3853] 🐛 fix(range): ajout de la possibilité de changer la value en JS [DS-3853] Oct 23, 2024
@@ -1,6 +1,7 @@
import api from '../../api.js';
import { RangeEmission } from './range-emission.js';
import { RangeConstraints } from './range-constraints';
import { completeAssign } from '../../../../core/script/api/utilities/property/complete-assign.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

il faut récupérer depuis api

Suggested change
import { completeAssign } from '../../../../core/script/api/utilities/property/complete-assign.js';

}
};

return completeAssign(super.proxy, proxyAccessors);
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return completeAssign(super.proxy, proxyAccessors);
return api.internals.property.completeAssign(super.proxy, proxyAccessors);

@keryanS keryanS requested a review from lab9fr October 25, 2024 13:29
@lab9fr lab9fr added Prêt and removed à valider labels Oct 25, 2024
@keryanS keryanS merged commit f15db8f into dev-1.13.0 Oct 28, 2024
4 checks passed
@keryanS keryanS deleted the feat/range-accessor-value branch October 28, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
correctif Quelque chose ne fonctionne pas maj 1.12.1 Prêt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants