Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

String sTime = "";
for (Object o : activityItems) {
List<Activity> activityList = (List<Activity>) activityItems.get(0);
for (Object o : activityList) {
Copy link
Member

Choose a reason for hiding this comment

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

erm, wouldn't make sense to then use Activity right away since you casted to a List of Activities already? ;)

Copy link
Member Author

Choose a reason for hiding this comment

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

It was a really quick fix to get it working.
But currently I am unsure what the nextcloud/android-library#73 pr is doing at all... ;-)

Copy link
Member

Choose a reason for hiding this comment

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

We should also find the root cause of #1848 since that reported bug would still happen here and cause a crash of the app...

Copy link
Member

Choose a reason for hiding this comment

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

Or not, I am not sure at this time of the day... too late at night...

@tobiasKaminsky
Copy link
Member Author

Postponed until clarification of library PR.

#1848 should be independent of this.

@tobiasKaminsky tobiasKaminsky added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Oct 11, 2018
@tobiasKaminsky
Copy link
Member Author

Closing in favor of #3130

@tobiasKaminsky tobiasKaminsky deleted the activityPreviews branch December 10, 2018 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing needs info Waiting for info from user(s). Issues with this label will auto-stale.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants