Moving . outside )

[djbrooke]

quick test for @qqmyers and @pdurbin of the "Allow edits from maintainers" feature. It's checked.

[qqmyers]

The change I made, which hopefully is in @djbrooke 's repo, was by clicking the edit button in the github GUI and selecting to commit directly to the branch. Does it look like it worked as expected?

[pdurbin]

@qqmyers yes, as expected, if we look at @djbrooke 's fork at https://github.com/djbrooke/dataverse-previewers/commits/patch-1 we can see your commit there (above his since it's newer). Here's a screenshot:

[qqmyers]

OK - and then one by cloning his repo and committing from my desktop using eclipse. (I did tell eclipse who I was at github so presumably it is sending my creds to allow me to upload to the djbrooke repo.

I think this is useful... Is this what didn't work with the QDR-IQSS Dataverse pulls?

[qqmyers]

One more note - if I go to the djbrooke fork and select a file in the patch-1 branch, the github pencil icon says 'edit this file' whereas if I select from the master branch, it says 'fork and edit', so I appear to only have access to this branch as I should.

[pdurbin]

What didn't work for me is what I wrote about at http://guides.dataverse.org/en/4.17/developers/version-control.html#adding-commits-to-a-pull-request-from-a-fork which boils down to the moral equivalent of what I wrote there:

git remote add OdumInstitute [email protected]:OdumInstitute/dataverse.git
git fetch OdumInstitute
git checkout 4709-postgresql_96
vim path/to/file.txt
git commit
git push OdumInstitute 4709-postgresql_96

To be clear, this is not a problem! When the above fails I just take it to mean that the contributor doesn't want anyone messing with their fork, which is a perfectly reasonable stance. In that case I just make a pull request into their pull request.

[qqmyers]

But I did leave the 'allow edits' option checked (should be true for all my PRs to Dataverse), so I'm confused as to why it doesn't work. (If that does work, then I'm confused about what the issue was on the last MDC PR)