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

Try pt-pt image fix #201

Merged
merged 2 commits into from
Aug 6, 2019
Merged

Try pt-pt image fix #201

merged 2 commits into from
Aug 6, 2019

Conversation

m52go
Copy link
Contributor

@m52go m52go commented Aug 6, 2019

Hard to test locally, as images work fine locally, and I don't have access to Netlify. Will revert this pull request if it doesn't work.

@m52go m52go merged commit fe370cd into bisq-network:master Aug 6, 2019
@m52go
Copy link
Contributor Author

m52go commented Aug 6, 2019

It worked! @huey735

@huey735
Copy link
Contributor

huey735 commented Aug 6, 2019

Just to be sure, does this mean that for the future we need to rename the image files and the urls inside dao_content_tr.yml?

@m52go
Copy link
Contributor Author

m52go commented Aug 6, 2019

Yes, each image for each language needs to have a filename that's different from that of the same image in every other language (e.g., just append the image file with the language code).

To be honest, I'm not sure it's worth the effort. Feel free to change them all back English if you like. I don't think it's a big deal. The images are meant to be visual anyway, so I'm sure people could get the main idea from the illustrations.

But that's the way to make it functional, if you'd like to stick with it.

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.

2 participants