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

[16.0] [IMP] account_statement_import_file: file_import_oca's label to 'OCA Import' #750

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

rrebollo
Copy link

@rrebollo rrebollo commented Dec 17, 2024

Related to #610.

In EE 16.0, we observe scenarios like:

image

This makes it impossible for non-technical users to distinguish which import method originates from OCA.

This PR proposes prefixing current label with "OCA". I think translations should also be updated accordingly.

@OCA-git-bot
Copy link
Contributor

Hi @etobella, @alexis-via,
some modules you are maintaining are being modified, check this out!

@rrebollo rrebollo force-pushed the 16.0-imp-account-statement-import-file-bank-statements-available-resources-label branch from b8f4588 to df0265b Compare December 17, 2024 00:25
@rrebollo
Copy link
Author

Failing tests not related to modified addon so ❓
@jorgeglez1990 @AinohaBH can U please make functional review?

@rrebollo rrebollo marked this pull request as ready for review December 17, 2024 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants