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: Allow registration of load child views callback on view #1046

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

Pytal
Copy link
Contributor

@Pytal Pytal commented Aug 8, 2024

New loadChildViews method added to be called to load child views

@Pytal Pytal added enhancement New feature or request 3. to review labels Aug 8, 2024
@Pytal Pytal requested review from susnux and skjnldsv August 8, 2024 04:10
@Pytal Pytal self-assigned this Aug 8, 2024
@Pytal Pytal enabled auto-merge August 8, 2024 04:10
Copy link

codecov bot commented Aug 8, 2024

Bundle Report

Changes will increase total bundle size by 416 bytes ⬆️

Bundle name Size Change
@nextcloud/files-esm 117.38kB 208 bytes ⬆️
@nextcloud/files-esm-cjs 118.77kB 208 bytes ⬆️

Copy link
Contributor

@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.

Sure! 👍

lib/navigation/view.ts Outdated Show resolved Hide resolved
lib/navigation/view.ts Outdated Show resolved Hide resolved
@sorbaugh
Copy link

sorbaugh commented Aug 8, 2024

Thanks for your reviews! 🙏 Do we know what the issue with the failing tests are?

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal changed the title feat: Allow registration of toggle callback on view feat: Allow registration of load child views callback on view Aug 8, 2024
@Pytal
Copy link
Contributor Author

Pytal commented Aug 8, 2024

Renamed to loadChildViews as this needs to be called if a view is already expanded not only when the user manually toggles it open

@Pytal Pytal merged commit b3561df into main Aug 8, 2024
15 checks passed
@Pytal Pytal deleted the feat/view-on-toggle-open branch August 8, 2024 17:50
@Pytal Pytal mentioned this pull request Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants