-
Notifications
You must be signed in to change notification settings - Fork 115
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
Update Scott Logic logo #1184
Update Scott Logic logo #1184
Conversation
|
✅ Deploy Preview for fdc3 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
@joelwilliamsmith @ColinEberhardt we'll need Joel to hit the Please click here to be authorized link from the EasyCLA notification to merge this, otherwise LGTM. Shout if you need any help.
Hi @kriswest - I've added @joelwilliamsmith to our 'corporate' CLA. |
/easycla |
@ColinEberhardt thanks, however @joelwilliamsmith will still need to pop back here and hit the Please click here to be authorized link (here or in the EasyCLA comment). This time it should authorize him which will allow us to merge the PR. This only needs doing once, thereafter it should pick up his authorized status on future PRs. Let us know if it fails to authorize (as that would indicate the wrong CLA was signed - FDC3 has a different one to other FINOS projects due to the Community Specification License) |
/easycla |
/easycla |
@ColinEberhardt Closing due to inactivity. Presumably, you still want the change made, could someone recreate the PR? Otherwise re-open if you can get @joelwilliamsmith to hit the authorisation link in the EasyCLA comment. |
No description provided.