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

Adding europe asia button section to osf homepage #531

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

Conversation

PeteShine1
Copy link
Contributor

Asia and Europe site buttons added above news section.

@smarcet
Copy link
Contributor

smarcet commented Dec 5, 2023

@PeteShine1 .env.template should not be deleted is a reference

@smarcet
Copy link
Contributor

smarcet commented Dec 5, 2023

@PeteShine1 also src/content/settings.json should be removed from the PR

@smarcet smarcet self-requested a review December 5, 2023 13:32
@smarcet
Copy link
Contributor

smarcet commented Dec 5, 2023

@PeteShine1 .python-version should be added to .gitignore and removed from this PR

Copy link
Contributor

@smarcet smarcet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PeteShine1 please review comments

@smarcet
Copy link
Contributor

smarcet commented Dec 5, 2023

src/pages/ptg/index.md Outdated Show resolved Hide resolved
@PeteShine1
Copy link
Contributor Author

In this last PR, I also took .env and .env* out of .gitignore, in an attempt to resolve #531 (comment)

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.

None yet

2 participants