Drop the Long Slave globals
Followup to !7030 (merged)
edit:
- Changed the global
V.eventDescription
to be an argument forlongSlave()
- Did the same for
V.saleDescription
, but changed the name tomarket
and it is expected to be a string if true now. - Removed
V.applyLaw
entirely. Replaced it with a function that checks themarket
string to see if it's an honest slave market. - Moved
slave
parameter outside of the brackets, as it is often used by itself.
New format is App.Desc.longSlave(slave, {market: "corporate", eventDescription: 0})
Edited by lowercasedonkey
Merge request reports
Activity
added 1 commit
- c70508e9 - change param saleDescription name to "market"
added 3 commits
- Resolved by lowercasedonkey
@svornost does the
includeDOM
widget require using the V proxy? I'm seeing both that and SC in the code and I've never seen that before. $ is rare though, wondering if it's wrong.Edited by lowercasedonkey
mentioned in merge request !7030 (merged)
added 30 commits
-
afc2fd2e...ab9ad728 - 29 commits from branch
pregmodfan:pregmod-master
- 5f724c48 - Merge branch 'pregmod-master' into final-ls
-
afc2fd2e...ab9ad728 - 29 commits from branch
Please register or sign in to reply