-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added CASLS retainer agreement #15
Conversation
…created and edited necessary attachment and object blocks. document is automating without issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work here.
A few things:
-
Down the CASLS letter path only, after collecting signatures, it collects addresses which are not used on output document. I don't think this should be happening. Maybe try looking into why these are being triggered?
…header and footer). could not resolve order bug and improper gather of client + atty address.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I must have totally not tested this - sorry for committing broken code. I was trying to fix the ordering and gather problem (it's gathering the address's for attorney and client after signatures and without me asking for it in any code I wrote). I tried to do it by changing how complete_attribute for attorneys and clients was implemented. I used the [i] based on how the process was discussed in the documentation here. I tried to just have complete_attribute defined only within the individual .yml but it didn't fix the gathering issue. |
No worries! You're working on an interview order block, and the link you sent is a question block (which is why they are working differently for you). Check out how I've edited the example you're looking at to do something similar to what you're going for. You can run this in the playground by creating a new project, and naming the yml file.
For troubleshooting why its asking for address even when the interview order and the template do not ask for it, on the page it asks for it, select developer information and scroll to the bottom and see why it's being called. A zip code might be required on download in AL, but we can figure it out if this is causing your issue. |
Thanks! I think it is about the download in AL. It calls CASLS_Retainer_Agreement['final'] and then looks for the clients address set that includes zip. For each variable in the set the interview would look for the object with "_label" after. So it looked for clients[0].address.address and pulled up the question block that gathers that, then it looked for clients[0].address.address_label. Then it moved to another variable from the set doing the same thing, ending on clients[0].address.zip_label. Not sure if this is helpful or makes sense. |
Can you commit the code that was getting this error? |
Just committed! The address gather problem also happens when you try to do other documents, see the no contact letter for a quick example. In the CASLS Retainer there is also a problem where it seems like it is gathering legal_problems before clients[0].name.full despite it being after the name gather. Or more specifically, legal_problems use of ${ clients[0].name } is what triggers the name gather. If you click the dev mode when it gathers name it makes it seem like legal_problems is what triggers the question - not the main order block. |
@nonprofittechy Is there a way to disable the collection of an address on download or preset it in AL? |
Yes, look for the store_variable_snapshot line that is calling for users[0].address.zip |
That's what I thought it was too, but couldn't find it when I looked. Will try again |
You can also use the icon in the playground to browse through mentions of any reference to users[0].address.address, and of course try the developer information button |
I did that for |
al_visual should be fine for this package. It definitely doesn't mention users[0].address.address Possibly this could be coming from the work @tobyfey was doing on the legal server integration, but I can't think there's anything in the Assembly Line that would force the address to get gathered. I think the court gathering used to trigger address, but I don't think it does for a year or more. |
With GBLS, I also had users[0].address.address being called when it wasn't in the interview or the templates. It would be called during document production, so I assumed it was in the documents, and later realized it wasn't (after I added a code block that set it to GBLS main office). |
This PR adds the docx and yml for the CASLS retainer agreement. It also makes necessary adjustments to attachment, objects, and bundles to produce document as output.
I also removed the attachment block from one of the yml for the plain_langauge_retainer per the Lemma Style Guide.