You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Good idea 💡 and not against, we just have to keep it true with Power Automate, since it is the core base - but we should still allow other use cases to be easy.
This could be combined with #47 to replace the current way functions are registered and 'retrieved', added a metadata class with shorthands, i.e. eq => equal, could be a good solution.
The use of shorthands should be controlled as an option, so it can be disregarded, and an error can be thrown, if ExpressionEngine is being used to test Power Automate flows.
I don't think we should bloat the current way functions are registered and retrieved, but instead recreate it properly :D
var services = new ServiceCollection();
services.addExpressionEngine();
services.AddFunctionMap<MyFunctionMap>();
----
class MyFunctionMap : IFunctionMapper
{
public string ResolveFunctionName(string name){
//logic to map names
}
}
class DefaultFunctionMap : IFunctionMapper
{
public string ResolveFunctionName(string name) => name;
}
Would be a none breaking feature but provide a way to do it.
Activity
thygesteffensen commentedon Dec 10, 2021
Good idea 💡 and not against, we just have to keep it true with Power Automate, since it is the core base - but we should still allow other use cases to be easy.
This could be combined with #47 to replace the current way functions are registered and 'retrieved', added a metadata class with shorthands, i.e. eq => equal, could be a good solution.
The use of shorthands should be controlled as an option, so it can be disregarded, and an error can be thrown, if ExpressionEngine is being used to test Power Automate flows.
I don't think we should bloat the current way functions are registered and retrieved, but instead recreate it properly :D
pksorensen commentedon Dec 10, 2021
We agree,
i am thinking
Would be a none breaking feature but provide a way to do it.
feat!: Redesigned the way functions are resolved
github-actions commentedon Mar 18, 2022
🎉 This issue has been resolved in version 4.0.0-dev.2 🎉
The release is available on:
v4.0.0-dev.2v4.0.0-dev.2Your semantic-release bot 📦🚀
github-actions commentedon Aug 23, 2022
🎉 This issue has been resolved in version 4.0.0 🎉
The release is available on:
v4.0.0v4.0.0Your semantic-release bot 📦🚀