- Issue created by @yas
- Merge request !50Issue #3468481 by yas: Add the artifacts (CFn) for the deployments for a Façade site → (Closed) created by yas
- Merge request !51Issue #3468481 by yas: Add the artifacts (CFn) for the deployments for a Façade site → (Merged) created by yas
- Status changed to Needs review
4 months ago 8:44am 16 August 2024 - 🇺🇸United States yas California 🇺🇸
@baldwinlouie
Can you please review the patch? Thanks!
- Status changed to Needs work
4 months ago 9:39pm 20 August 2024 - 🇺🇸United States baldwinlouie
@yas, thank you for the patch. I just have these minor comments. Aside from that, it looks good.
- Status changed to Needs review
4 months ago 10:37pm 20 August 2024 - 🇺🇸United States yas California 🇺🇸
@baldwinlouie
Thank you for your review. I found more typos and fixed them. Can you please review the patch again? Thanks!
- Status changed to RTBC
4 months ago 12:08am 21 August 2024 - 🇺🇸United States baldwinlouie
@yas, thank you for the update on the patch! It looks good to me now.
- 🇺🇸United States yas California 🇺🇸
@baldwinlouie
Thank you for your review. I’ll merge the patch into
2.0.x
; and close this issue as Fixed. - Status changed to Fixed
4 months ago 12:39am 21 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.