Closed xmulligan closed 3 years ago
@xmulligan Is this ready to be merged then?
@edwarnicke Yes I think it is ready
Yes it’s ready
On Wednesday, May 5, 2021, Ed Warnicke @.***> wrote:
@xmulligan https://github.com/xmulligan Is this ready to be merged then?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inclusivenaming/website/pull/57#issuecomment-832907287, or unsubscribe https://github.com/notifications/unsubscribe-auth/APHMGWMHYKF6HXZVYXMBWI3TMGD5DANCNFSM435KJNQA .
@xmulligan We have a bit of a mechanical problem (see https://deploy-preview-57--inclusivenaming.netlify.app/ )... img src was wrong. I've suggested what I think is the right thing. If so, please correct and we can proceed to merge :)
Thanks @edwarnicke it works now. appreciate the help
/lgtm
@xmulligan We all get by with a little help from our friends... I've made many many many such small errors in my day, and had most of them caught by friends in code review :) Just paying it forward here :)
@shahed-al I found where to put the link