Skip to content

Sidebar sections integration - #16312

Closed
juliusknorr wants to merge 3 commits into
enhancement/sidebar/standardfrom
sidebar-integration
Closed

Sidebar sections integration#16312
juliusknorr wants to merge 3 commits into
enhancement/sidebar/standardfrom
sidebar-integration

Conversation

@juliusknorr

@juliusknorrjuliusknorr commented Jul 9, 2019

Copy link
Copy Markdown
Member

Requires a new build of nextcloud-vue-collections

skjnldsvand others added 3 commits July 8, 2019 18:27
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
<div :is="section($refs['section-'+$index], fileInfo)" :file-info="fileInfo"></div>
</div>

<collection-list v-if="fileInfo" type="file" :id="`${fileInfo.id}`" :name="fileInfo.name"></collection-list>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you need it as string?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Yes, the id of a project resource is of type string, so we need to cast.

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

@skjnldsv I leave bumping the nextcloud-vue-collections to 0.5.4 up to you if thats fine, so we have a bit less rebasing work here.

@skjnldsv
skjnldsvforce-pushed the enhancement/sidebar/standard branch 3 times, most recently from 7b7239c to 87480c0CompareJuly 12, 2019 19:00
@MorrisJobkeMorrisJobke mentioned this pull request Jul 15, 2019
28 tasks
@MorrisJobke

Copy link
Copy Markdown
Member

@juliushaertl What is the status here? We are close to the beta 1. Should this go into 17 or 18?

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

18 since this is part of #15719

@skjnldsv
skjnldsvforce-pushed the enhancement/sidebar/standard branch 4 times, most recently from 9cf8980 to e45019fCompareJuly 28, 2019 08:53
@skjnldsv
skjnldsvforce-pushed the enhancement/sidebar/standard branch from 3bcc7b6 to 4241a55CompareAugust 8, 2019 17:22
@skjnldsv
skjnldsvforce-pushed the enhancement/sidebar/standard branch from 866e069 to f27f965CompareSeptember 9, 2019 08:58
@skjnldsv
skjnldsvforce-pushed the enhancement/sidebar/standard branch from 00ff19d to cc1a2c9CompareSeptember 16, 2019 19:04
@skjnldsv
skjnldsv deleted the sidebar-integration branch September 16, 2019 23:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@juliusknorr@MorrisJobke@skjnldsv