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

task #6 changed blog to bilingual #33

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

Conversation

Mublin
Copy link

@Mublin Mublin commented Oct 21, 2022

what I did

created a docs folder

created a ar folder in the docs

added a nav item in _quarto.yml file in the root directory (arabic and it's href to ar(ar directory) )

added another property in _quarto.yml (output-dir: docs) which changed the overriding directory when index.qml is rendered

the _quarto.yml now makes the _site folders in the docs directory rather than the root directory

@netlify
Copy link

netlify bot commented Oct 21, 2022

Deploy Preview for oscsa-en-blog failed.

Name Link
🔨 Latest commit 1ab6e8d
🔍 Latest deploy log https://app.netlify.com/sites/oscsa-en-blog/deploys/638f0ecbdf59d2000895091a

@okerekechinweotito
Copy link
Contributor

@Mublin Can you make a new PR ? This would enable the mentors @BatoolMM and @alswajiab to see your changes in the Deploy-preview. Also you changed too many files. I would suggest you break down your changes into smaller parts

@Mublin
Copy link
Author

Mublin commented Oct 24, 2022

@ba

@Mublin Can you make a new PR ? This would enable the mentors @BatoolMM and @alswajiab to see your changes in the Deploy-preview. Also you changed too many files. I would suggest you break down your changes into smaller parts

I cannot because to add the arabic version I have to add the arabic files but I will try a new PR

@Mublin
Copy link
Author

Mublin commented Oct 24, 2022

Can I ask you a question @BatoolMM and @alswajiab

@BatoolMM
Copy link
Member

BatoolMM commented Nov 3, 2022

@Mublin sorry, what was your question?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants