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

SWIK-2257 Show recently used decks on the homepage if logged in #888

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

Conversation

rmeissn
Copy link
Member

@rmeissn rmeissn commented Jun 6, 2018

This commit will hide the carousel and show recently edited decks instead, like on the My Decks page, if a user is logged into SlideWiki. This has been requested by some trial partners.

Not sure this is the best way to implement it - especially about the new action.

EDIT: This PR won't cover SWIK-2257 entirely but is a first step towards it. @abijames Shall this PR be closed and opend up again as soon as the mockup from Jira is implemented or is it worth to merge this already? It might be useful to show as of deliverable 3.6.

@rmeissn rmeissn self-assigned this Jun 6, 2018
@rmeissn rmeissn requested review from abijames and kadevgraaf June 6, 2018 09:19
Copy link
Member

@kadevgraaf kadevgraaf left a comment

Choose a reason for hiding this comment

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

Looks good!

@kadevgraaf
Copy link
Member

@abijames needs to review this Before merging.

@rmeissn rmeissn changed the title Show recently used decks on the homepage if logged in SWIK-2257 Show recently used decks on the homepage if logged in Aug 2, 2018
@kprist kprist assigned kprist and unassigned rmeissn Oct 19, 2018
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