- πΊπΈUnited States dpagini
Can this be reviewed for Lockr 5.x if the same config objects are supported in that branch?
- Assigned to jlockhart
- π¨π΄Colombia jlockhart Nomadic: Colombia
I might need more information here. Based on what I'm seeing Lockr does provide lockr.lockr_secret.* in the lockr.schema.yml file. See image. Are we missing schema definition somewhere else?
- πΊπΈUnited States dpagini
The error I get is something along the lines of
lockr.lockr_secret._____:info.wrapping_key missing schema
. In the code in your screenshot, you are setting `info` to a sequence, but then not defining that sequence. I don't think you want to define this key as a sequence, but instead as a mapping, with the corresponding keys (wrapping_key) then defined.
Did you get the file that I sent over to Chris T? That has the changes that are working for our team, but I don't think you want the patch exactly. - @jlockhart opened merge request.
- @jlockhart opened merge request.
- @cellar-door opened merge request.
- πΊπΈUnited States dpagini
I'm still seeing failures with this commit. It looks to me like there's an indentation issue in the YML file. I would recommend setting up a very basic automated test in the module that enables a test module that has this config present, and then you should be able to see the schema errors for yourself.
- @cellar-door opened merge request.
- πΊπΈUnited States dpagini
Confirming this issue is fixed in the latest `dev-5.x` branch. Thank you all!