-
-
Notifications
You must be signed in to change notification settings - Fork 547
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
[17.0][MIG] base_import_security_group #1010
Open
StefanRijnhart
wants to merge
28
commits into
OCA:17.0
Choose a base branch
from
StefanRijnhart:17.0-mig-base_import_security_group
base: 17.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[17.0][MIG] base_import_security_group #1010
StefanRijnhart
wants to merge
28
commits into
OCA:17.0
from
StefanRijnhart:17.0-mig-base_import_security_group
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This module makes importing data from CSV files optional for each user, allowing it only for those users belonging to a specific group. [FIX] Solved flake8 error E305: expected 2 blank lines after class or function definition, found 1 [FIX] Renamed base_import_csv_optional in base_import_security_group [FIX] Solved flake8 error E501: line too long; [FIX] Removed unnecessary monkeypatch and extended Base.load method; [FIX] Bugfixing in Base.load method; [FIX] Code refactoring [IMP] Tests created
[FIX] Updated travis postgresql version [FIX] Updated phantom_js version in travis [FIX] Configured travis to run tests for this module in isolation mode [FIX] Configured travis to run tests for this module in isolation mode [FIX] Added group to demo_user for test purposes [FIX] Changed waiting condition on Phantom JS test [FIX] Removed Travis configuration for isolated test environment [FIX] Changed waiting condition in phantom_js test [FIX] Changed target action in phantom_js test [FIX] Changed target action in phantom_js test
OCA Transbot updated translations from Transifex [UPD] Update base_import_security_group.pot [UPD] Update base_import_security_group.pot [UPD] README.rst
[UPD] Update base_import_security_group.pot [UPD] README.rst
* Patch existing class instead of duplicating it, for better compatibility with other modules enhancing the same class. * To circumvent problems executing async calls in a non-widget class, pass the user's import permission through the web session. * Adapt tests to consult he Favorite dropdown in the 14.0 GUI. * In the tour, don't target the Users menu which is the default menu of the Settings menu when no other module is installed, and in that case clicking the menu in the tour is a void action that makes the tests fail. Target the Companies menu item instead. [UPD] Update base_import_security_group.pot [UPD] README.rst
[UPD] Update base_import_security_group.pot [UPD] README.rst
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-ux-15.0/server-ux-15.0-base_import_security_group Translate-URL: https://translation.odoo-community.org/projects/server-ux-15-0/server-ux-15-0-base_import_security_group/
When the web_responsive module is installed, the tests fail when looking for "Settings" in the menu, because you have to access the menu first to get that option. Adding the extra step to display the menu is the right option so that the tests don't fail whether the web_responsive module is installed or not. TT42822
Currently translated at 100.0% (3 of 3 strings) Translation: server-ux-16.0/server-ux-16.0-base_import_security_group Translate-URL: https://translation.odoo-community.org/projects/server-ux-16-0/server-ux-16-0-base_import_security_group/es/
Currently translated at 100.0% (2 of 2 strings) Translation: server-ux-16.0/server-ux-16.0-base_import_security_group Translate-URL: https://translation.odoo-community.org/projects/server-ux-16-0/server-ux-16-0-base_import_security_group/it/
StefanRijnhart
force-pushed
the
17.0-mig-base_import_security_group
branch
from
January 29, 2025 21:32
3b175d7
to
34b5de6
Compare
/ocabot migration base_import_security_group |
StefanRijnhart
force-pushed
the
17.0-mig-base_import_security_group
branch
from
January 30, 2025 15:26
34b5de6
to
5a47dff
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JS tour test: migrate to OWL and update selectors