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

[Regression Fix] Readd loading icon #479

Merged
merged 3 commits into from
Oct 14, 2020

Conversation

Mikescops
Copy link
Member

Fix for regression added by #468.

With the last vue-virtual-grid update we can set a loader component until the next batch is loaded.

Signed-off-by: Corentin Mors <corentin.mors@dashlane.com>
@Mikescops Mikescops added bug Something isn't working 3. to review Waiting for reviews labels Oct 13, 2020
@Mikescops Mikescops requested a review from skjnldsv October 13, 2020 14:56
@Mikescops Mikescops self-assigned this Oct 13, 2020
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Minor change request, good otherwise

Signed-off-by: Corentin Mors <corentin.mors@dashlane.com>
Comment on lines 2 to 4
- @copyright Copyright (c) 2020 Nextcloud
-
- @license GNU AGPL version 3 or any later version
Copy link
Member

Choose a reason for hiding this comment

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

Nope, you're the owner!

Copy link
Member

Choose a reason for hiding this comment

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

Btw please also add yourself as the author for the other files you edited

Copy link
Member Author

Choose a reason for hiding this comment

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

😞

Copy link
Member Author

Choose a reason for hiding this comment

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

done

Signed-off-by: Corentin Mors <corentin.mors@dashlane.com>
@skjnldsv skjnldsv merged commit 23c8569 into platform/readd-timeline-caching Oct 14, 2020
@skjnldsv skjnldsv deleted the platform/readd-loading-icon branch October 14, 2020 11:12
@Mikescops
Copy link
Member Author

@skjnldsv it was not a merge on master...

Need to review #478 first

@skjnldsv
Copy link
Member

I thought it was on purpose so I could review 478 after 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants