Conversation
|
@briri I think we need to discuss this. Maybe on Monday? Most of this looks fine from what I can tell but I am concerned about the use of "is_other" and "other_organisation" because I think we may be using these differently from you. At least, specifically: Let's talk about it Monday. Cheers. |
|
ok @raycarrick-ed if those two fields are still used heavily in your installations then I can revert those. The switch over from is_other to managed was part of the move to use the ROR API which I know has been problematic. Happy to chat about it more on Monday. |
Fixes #2818
Changes proposed in this PR:
Exceptions:
settingstable. We fixed a bug with theexported_planstable so I think it is still used. Not sure why we would want to retain the selected font and margins but 🤷🏻♂️research_outputsand removing themime_typestable