localgovdrupal / docs

Documentation for all projects around LocalGovDrupal.
https://trusting-noyce-aebebc.netlify.app/
GNU General Public License v2.0
6 stars 19 forks source link

Update terms-and-conditions.md #189

Closed aaronhirtenstein closed 1 year ago

aaronhirtenstein commented 1 year ago

updated the T&Cs to include info about membership and link to new contributor agreement

netlify[bot] commented 1 year ago

Deploy Preview for inspiring-euclid-d918c8 ready!

Name Link
Latest commit 6fb2e1148d51bddaa4d5d32136d55bd5dbc8fe70
Latest deploy log https://app.netlify.com/sites/inspiring-euclid-d918c8/deploys/647f0f752e6cec0008316773
Deploy Preview https://deploy-preview-189--inspiring-euclid-d918c8.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

msayoung commented 1 year ago

Easier to review here: https://deploy-preview-189--inspiring-euclid-d918c8.netlify.app/terms-and-conditions.html

aaronhirtenstein commented 1 year ago

Thanks for spotting those @msayoung how do I make those changes to this commit??

aaronhirtenstein commented 1 year ago

OK think I figured it out. Back to you @msayoung

msayoung commented 1 year ago

Approved. Out of interest, can you merge now @aaronhirtenstein ?

finnlewis commented 1 year ago

I was about to hit merge, but saw your comment @msayoung

@aaronhirtenstein are you able to merge your pull request, even though it came from a forked repo rather than a branch on this one?

finnlewis commented 1 year ago

For the record, it sounds like Aaron cannot merge in this case, as the pull request came from another organisation.

it looks like @aaronhirtenstein is a member of the organisation and does have write access to this repository, so maybe this is just because it was a fork to another repo.

See https://github.com/orgs/localgovdrupal/people/aaronhirtenstein

I will check this workflow again with another account.