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

[17/10/2022]Tasks 3.1: added event subpage link on navbar(English and Arabic pages) #180

Conversation

sandygudie
Copy link
Member

@sandygudie sandygudie commented Oct 16, 2022

Summary: issue 179

Tasks 3:
We need to add an events subpage where we can link our events in Zenodo.
3.1 Add this to the nav bar.

Fixes

  • Added event subpage link on navbar*
  • The event link is added to the navbar of the English and Arabic pages.
    image

What should a reviewer concentrate their feedback on?

  • Everything looks ok?

@netlify
Copy link

netlify bot commented Oct 16, 2022

Deploy Preview for oscksa ready!

Name Link
🔨 Latest commit 3bda899
🔍 Latest deploy log https://app.netlify.com/sites/oscksa/deploys/6356493f56f57100080ca435
😎 Deploy Preview https://deploy-preview-180--oscksa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@sandygudie sandygudie changed the title added event subpage link on navbar(English and Arabic pages) Tasks 3: added event subpage link on navbar(English and Arabic pages) Oct 17, 2022
@sandygudie sandygudie changed the title Tasks 3: added event subpage link on navbar(English and Arabic pages) [17/10/2022]Tasks 3: added event subpage link on navbar(English and Arabic pages) Oct 18, 2022
@sandygudie sandygudie changed the title [17/10/2022]Tasks 3: added event subpage link on navbar(English and Arabic pages) [17/10/2022]Tasks 3.1: added event subpage link on navbar(English and Arabic pages) Oct 18, 2022
@sandygudie
Copy link
Member Author

@alswajiab @BatoolMM please review

@sandygudie
Copy link
Member Author

Closing this Issue because it's completed in #187

@sandygudie sandygudie closed this Oct 31, 2022
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