Skip to content

Conversation

@MuditJain5
Copy link
Contributor

Fixes #6800

Changed the order of left side menu

The new order :

  • Info
  • Tickets
  • Schedule
  • Speakers
  • Calendar
  • Sponsors
  • Exhibition
  • Organizer
  • Call for Speakers
  • Getting here
  • Code of Conduct

Checklist

  • I have read the Contribution & Best practices Guide.
  • My branch is up-to-date with the Upstream development branch.
  • The acceptance, integration, unit tests and linter pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@vercel
Copy link

vercel bot commented Feb 23, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/2GeSqM6MYnzZQ9fHmKsG28hR2QSy
✅ Preview: https://open-event-frontend-git-fork-muditjain5-issue-6800-eventyay.vercel.app

@codecov
Copy link

codecov bot commented Feb 23, 2021

Codecov Report

Merging #6804 (b349151) into development (88da9db) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #6804   +/-   ##
============================================
  Coverage        21.44%   21.44%           
============================================
  Files              552      552           
  Lines             6215     6215           
  Branches           139      139           
============================================
  Hits              1333     1333           
  Misses            4856     4856           
  Partials            26       26           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88da9db...b349151. Read the comment docs.

Copy link
Contributor

@Naman-1234 Naman-1234 left a comment

Choose a reason for hiding this comment

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

Try to provide a screenshot, Whenever you deal with issues related to designing or reordering etc. This makes it easy for maintainers to review.

@MuditJain5
Copy link
Contributor Author

Try to provide a screenshot, Whenever you deal with issues related to designing or reordering etc. This makes it easy for maintainers to review.

Sorry, but I can't find all the menu items together in any event that's why I didn't provide any screenshot.

@afzal442
Copy link
Contributor

@MuditJain5 lookup here https://open-event-frontend-git-fork-muditjain5-issue-6800-eventyay.vercel.app/e/fa96ae2c
and share the screenshot here as well.

@MuditJain5
Copy link
Contributor Author

@MuditJain5 lookup here https://open-event-frontend-git-fork-muditjain5-issue-6800-eventyay.vercel.app/e/fa96ae2c
and share the screenshot here as well.

Thanks @afzal442 for helping me

@MuditJain5
Copy link
Contributor Author

Screenshot of the new order of Left Side Menu :

issue-6800

Copy link
Member

@mariobehling mariobehling left a comment

Choose a reason for hiding this comment

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

Works for me. Thanks!

@iamareebjamal iamareebjamal merged commit 71a6ecb into fossasia:development Feb 28, 2021
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.

Public Event Page: Change order of left side menu

5 participants