Skip to content

Conversation

@edmundmiller
Copy link
Collaborator

Moved some stuff around. I'd like to make it pop a little bit more and direct people's eyes of wear to look like https://nf-co.re/join

Creating the PR looking for feedback!

@jakobnissen
Copy link
Member

Thanks for doing this.
I think it would be nice to have a short "contact" section on the main page, perhaps just pointing to Discourse and Slack. At least to me, if I had any questions about using BioJulia as a user, I might not look for contact info in the "Join us" page in the sidebar.

@kescobo
Copy link
Member

kescobo commented Aug 11, 2023

Build errors here and #21 appear to be problems with permissions. There's probably some secret thing that we can add to the build configuration, but for now I'm just going to add @emiller88 to the repo with write permission and run them again.

@kescobo
Copy link
Member

kescobo commented Aug 11, 2023

@kescobo kescobo added enhancement Content Anything to do with written content labels Aug 11, 2023
@tlienart
Copy link
Collaborator

the issue is something to do with tokens some notes here from 2020 on the github-pages-action repo; given that this PR: https://github.com/BioJulia/biojulia.github.io/actions/runs/5833801346 worked fine, can you try just merging stuff and seeing if it works?

What might happen is that the preview path is faulty (e.g. it tries to do something with a non-existing branch, I'd need to check) but the rest works fine.

@kescobo
Copy link
Member

kescobo commented Aug 11, 2023

Yeah, I can build it locally, and clearly it works on merging. I also works if the branch originates from this repo (eg BioJulia/biojulia.github.io#17 (comment)).

But it would be awesome to have a way for external collaborators to make contributions w/o write permissions and still get build previews. [Feature request] because you obviously don't have enough on your plate 😆

@tlienart
Copy link
Collaborator

No it's good to know that the preview path doesn't work properly, GitHub actions are very powerful but I'm still a beginner when it comes to those so im not surprised there are issues; definitely something to look into as your use case is one that I'd expect many people to have (eg the Julia website folks..)

@edmundmiller
Copy link
Collaborator Author

https://docs.github.com/en/actions/security-guides/automatic-token-authentication#permissions-for-the-github_token

When a workflow is triggered by the pull_request_target event, the GITHUB_TOKEN is granted read/write repository permission, even when it is triggered from a public fork. For more information, see "Events that trigger workflows."

This event runs in the context of the base of the pull request, rather than in the context of the merge commit, as the pull_request event does.

Let's see if this works

@edmundmiller
Copy link
Collaborator Author

Partial success. It runs, but still no preview URL 😞

@kescobo
Copy link
Member

kescobo commented Aug 13, 2023

🤔 well, that's a kind of progress...

@tlienart
Copy link
Collaborator

tlienart commented Aug 14, 2023

what may have happened is that on the last trigger the build action saw "no change" so that there was effectively "nothing new to commit". Could I suggest maybe trying to re-trigger with a dummy commit where you change a word or a . or something?

(I need to do some more thorough investigation of this preview path with external contributors but I don't really have the time atm to dig through that... sorry)

@edmundmiller
Copy link
Collaborator Author

Have we confirmed that it works from a non-fork branch as well?

@kescobo
Copy link
Member

kescobo commented Aug 14, 2023

Let's find out: BioJulia/biojulia.github.io#24

@kescobo
Copy link
Member

kescobo commented Aug 14, 2023

Nope :-( https://github.com/BioJulia/biojulia.github.io/actions/runs/5857819238/job/15880512464

@edmundmiller
Copy link
Collaborator Author

Cool, that's progress! Let's follow up in #23, and we can focus on the join page here now.

@edmundmiller edmundmiller marked this pull request as ready for review August 22, 2023 14:58
@edmundmiller
Copy link
Collaborator Author

It's a start, if there's any outstanding issues let me know and we can either address before merging this or follow up in another PR!

@kescobo
Copy link
Member

kescobo commented Aug 22, 2023

@emiller88 Can you resolve the conflict here? I can't remember which edit on the header came first / is preferred. I'm not picky

@edmundmiller
Copy link
Collaborator Author

Done!

@kescobo kescobo merged commit d6dc22a into BioJulia:master Aug 24, 2023
@edmundmiller edmundmiller deleted the join branch August 24, 2023 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content Anything to do with written content enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants