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

Add option to set admin specific scheme/host/port #684

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

CGillen
Copy link

@CGillen CGillen commented May 16, 2022

Fixes #610
@ashmaroli mentioned they wanted tests on the fix. These might not be the best tests, so if you have any suggestions I'll gladly fix them up.

@ashmaroli
Copy link
Member

ashmaroli commented May 16, 2022

@CGillen Thank you for proposing an enhancement.

From a quick glance, it looks like you're assuming site.config["jekyll_admin"] will always be defined.
What if it isn't?

@CGillen
Copy link
Author

CGillen commented May 16, 2022

#dig would be a better solution, wouldn't it? Doubtful site.config["jekyll_admin"] would be defined as anything besides a hash.

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.

Jekyll url is not applied to the View button
2 participants