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

feat: Provide access to features via Event Dashboard #1762

Merged
merged 1 commit into from
Jun 29, 2019
Merged

feat: Provide access to features via Event Dashboard #1762

merged 1 commit into from
Jun 29, 2019

Conversation

ShridharGoel
Copy link
Member

@ShridharGoel ShridharGoel commented Jun 28, 2019

Fixes #1761

Changes:

Added a box above the section of 'Gross Sales' in Event Dashboard, containing the following items:

  • Check In
  • Orders
  • Tickets
  • Edit Event
  • Event Settings

Orders, Tickets and Edit Event would show only in Developer Mode.

Screenshot for the change:

Provide access to features via Event Dashboard

@auto-label auto-label bot added the feature label Jun 28, 2019
@fossasia fossasia deleted a comment Jun 28, 2019
@fossasia fossasia deleted a comment Jun 28, 2019
@ShridharGoel ShridharGoel changed the title feat: Provide access to features from Event Dashboard feat: Provide access to features via Event Dashboard Jun 28, 2019
@iamareebjamal
Copy link
Member

iamareebjamal commented Jun 28, 2019

Change the color from blue to black/grey please. And implement as standard list items ensuring that clicks ripple etc

@ShridharGoel
Copy link
Member Author

@iamareebjamal Should I implement a RecyclerView or just wrap each layout in a CardView?

@iamareebjamal
Copy link
Member

No need of any of that. There is just an attribute to make it ripple

@ShridharGoel
Copy link
Member Author

ShridharGoel commented Jun 29, 2019

@iamareebjamal Updated. Actually, I asked that since you told to implement as standard list items.

@fossasia fossasia deleted a comment Jun 29, 2019
@iamareebjamal
Copy link
Member

Don't remove, nowhere in the parent issue is it mentioned to remove the navigation menu items

@fossasia fossasia deleted a comment Jun 29, 2019
iamareebjamal
iamareebjamal previously approved these changes Jun 29, 2019
@iamareebjamal iamareebjamal merged commit 88c0574 into fossasia:development Jun 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide access to mentioned features via Event Dashboard instead of navigation drawer
2 participants