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
I'm having trouble understanding the ACL functions which @webplusai has contributed to MWS.
The main problem I'm running into is that there are some inconsistencies in the acl code which typescript catches, but they appear to be actual bugs and not just typing problems.
Is the bags table missing an owner_id column? Some of the code seems to indicate that.
The ACL concept that I'm seeing seems pretty good. I'm just going through and converting everything into consistent and repeatable patterns that are easier to maintain.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm having trouble understanding the ACL functions which @webplusai has contributed to MWS.
The main problem I'm running into is that there are some inconsistencies in the acl code which typescript catches, but they appear to be actual bugs and not just typing problems.
Is the bags table missing an owner_id column? Some of the code seems to indicate that.
The ACL concept that I'm seeing seems pretty good. I'm just going through and converting everything into consistent and repeatable patterns that are easier to maintain.
extra notes
Diff between the PR branch and MWS
Diff in the opposite direction
PRs by webplusai
Commits by webplusai
Turns out the VSCode source control graph appears to be the most useful.
The main commits starting with the first ACL commit appear to be:
Based on that, it seems that the most important comparison is between the multiwikiserver plugin folder then and now.
handler files changed by acl (besides several adding useAcl and entityType)
created files
Beta Was this translation helpful? Give feedback.
All reactions