-
Notifications
You must be signed in to change notification settings - Fork 10
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
7.3 Deployment #470
7.3 Deployment #470
Conversation
@IsakNaslundBh, @Chrisshort92, @peterjamesnugent, I've tested and reviewed the code against sections 01 to 09 of the Test Procedure for Structural Adapters using the 240916 Alfa Installer. All good globally and happy with it. I've noted the main areas/items to be improved moving forward at the following link: Only bug found has been sorted and the related Pull Request (to be reviewed and approved) is at the link below: |
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.
Approved based on review by @GCRA101 , bug fix highlighted above has also been merged.
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.
PR has been checked again to test that recent bug fix has been adopted into this repo. Approved now that the functionality has been tested again with build on 19/09/2024
@BHoMBot check required |
@Chrisshort92 to confirm, the following actions are now queued:
|
The check |
The check |
The check |
The check |
The check |
The check |
The check |
@BHoMBot just to let you know, I have provided a |
ETABS_Toolkit
Feature
Compliance
PierForce
and removed inheritance ofBarForce
;