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

[15.0][FIX] dms: Fix smartbuttons from files and directories #293

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Feb 8, 2024

Use case when clicking on smartbutton Files in a directory:

  • Only files in that directory or subdirectories are shown.
  • Only directory and child subdirectories are displayed in search panel.
  • Directory is auto-selected in search panel.

ejemplo-carpetas

Use case when clicking on smartbutton Subdirectories in a directory:

  • Only directories in that directory or subdirectories are displayed.
  • Directory is auto-selected in search panel.

ejemplo-archivos

Please @pedrobaeza and @chienandalu can you review it?

@Tecnativa

@pedrobaeza
Copy link
Member

pedrobaeza commented Feb 8, 2024

You should change the first smart-button reference from Files to Subdirectories.

@pedrobaeza pedrobaeza added this to the 15.0 milestone Feb 8, 2024
@victoralmau victoralmau force-pushed the 15.0-fix-dms-smartbuttons branch from a62e06f to 00113b5 Compare February 8, 2024 11:24
@victoralmau
Copy link
Member Author

You should change the first smart-button reference from Files to Subdirectories.

Sorry, I don't understand exactly where you mean (kanban view or form view). I think the buttons are currently displayed: Subdirectories and Files

@pedrobaeza
Copy link
Member

I mean in the commit message/PR message.

Use case when clicking on smartbutton Files in a directory:
- Only files in that directory or subdirectories are shown.
- Only directory and child subdirectories are displayed in search panel.
- Directory is auto-selected in search panel.

Use case when clicking on smartbutton Subdirectories in a directory:
- Only directories in that directory or subdirectories are displayed.
- Directory is auto-selected in search panel.
@victoralmau victoralmau force-pushed the 15.0-fix-dms-smartbuttons branch from 00113b5 to 3107782 Compare February 12, 2024 07:08
@victoralmau
Copy link
Member Author

Changes done.

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-293-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 0a22bc0 into OCA:15.0 Feb 12, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 57a31f9. Thanks a lot for contributing to OCA. ❤️

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.

4 participants