Replies: 1 comment
-
The destination function was there for a reason as at one point it was different. I think maybe MVM removed the need for it though. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hey all,
Opening this to suggest some redundant/problematic APIs we might want to get rid of in v7.
MasteryService#getMasteryDataForDestination
This API is literally identical to
MasteryService#getMasteryData
- it just forwards the call there. This seems pointless to have.LogCategory#CALLER
Using
arguments.caller
is highly problematic, and doesn't even work in strict mode. It's also deprecated, see MDN.Let me know what everyone thinks.
Beta Was this translation helpful? Give feedback.
All reactions