Open isc-tleavitt opened 4 weeks ago
I don't think the %Installer stuff is being kept up to date. Focus has moved to cpf merge so I don't think the Installer stuff. I recommend re-writing the class to not use %Installer and directly wrap around Security.Applications which is the source of truth and always kept up to date. We could also consider packaging cpf merge files with the package manager and apply those when a package is installed (this could be tricky though since the instance needs to be restarted for this I believe).
"I don't think the %Installer stuff is being kept up to date." - that was my suspicion too, but you never know. Wrapping Security.Applications makes sense to me.
@isc-shuliu could you poke around at this a bit when you wrap up the CI? Would be good to have at least doc / a sample of how it works with IPM 0.7.0 by the end of next week.
Context: https://community.intersystems.com/post/intersystems-python-contest-2024
Doc should be in wiki and in a Developer Community post.
see https://docs.intersystems.com/irislatest/csp/docbook/DocBook.UI.Page.cls?KEY=GCRN_new20241#GCRN_new20241_dev_wsgi
The existing CSPApplication class should work (assuming %Installer is up-to-date with the new features), but we might want a new resource processor class to make this even easier.