search

ava-innersource / Liquid-Application-Framework-1.0-deprecated

Liquid is a framework to speed up the development of microservices
MIT License
25 stars 13 forks source link

fix(LightWorker): removes authorization #204

Closed bruno-brant closed 4 years ago

bruno-brant commented 4 years ago

As can be seen in the discussion at #198, Authorization has many issues that render the system unusable or at least defective.

Closes #198.

bruno-brant commented 4 years ago

I'm uncertain about removing LightMessage. I guess we may use it in the future if we create a more elaborated messaging system.... for that reason, I've not marked it as obsolete yet - and we can't simply remove it because our users are inheriting from it.

bruno-brant commented 4 years ago

@guilhermeluizsp as we discussed, I reverted the change - LightViewModel still uses CRTP.

guilhermeluizsp commented 4 years ago

@bruno-brant Everything is looking good! I'm just waiting for you to rebase with the master branch so I can finally approve it.

bruno-brant commented 4 years ago

@bruno-brant Everything is looking good! I'm just waiting for you to rebase with the master branch so I can finally approve it.

Done! It's hard to keep track of all the branches I must rebase whenever I rebase another. :)