Skip to content
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

Set url in site config (_config.yml) #1202

Closed
wants to merge 2 commits into from

Conversation

lrasmus
Copy link
Contributor

@lrasmus lrasmus commented Jun 27, 2023

Description

This should address issue #1201 where sharing links weren't working because site.url was blank (because url is blank in _config.yml).

Checklist:

  • I have previewed changes locally or with CircleCI (runs when PR is created)
  • I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

This should address issue USRSE#1201 where sharing links weren't working because site.url was blank (because url is blank in _config.yml).
@lrasmus
Copy link
Contributor Author

lrasmus commented Jun 28, 2023

Not as simple as I thought! Going to revisit and reopen a PR when I've got something that seems it'll work.

@lrasmus lrasmus closed this Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant