Skip to content

[15.0][MIG] pos_user_restriction: Migration to 15.0 #1010

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from Jul 4, 2023
Merged

[15.0][MIG] pos_user_restriction: Migration to 15.0 #1010

merged 12 commits into from Jul 4, 2023

Conversation

ghost
Copy link

@ghost ghost commented Jun 8, 2023

No description provided.

eLBati and others added 11 commits June 7, 2023 15:07
@eLBati
ADD pos_user_restriction
861cbeb 
Restrict some users to only access their assigned points of sale
@danimv5
Added translation using Weblate (Spanish)
1414ee1
@danimv5
Translated using Weblate (Spanish)
c4da422 
Currently translated at 100.0% (7 of 7 strings)

Translation: pos-12.0/pos-12.0-pos_user_restriction
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_user_restriction/es/
@eLBati
FIX pos_user_restriction allowing "User: Assigned POS Only" to see po…
99f71c6 
…int of sale menu
@OCA-git-bot
pos_user_restriction 12.0.1.0.1
9589764
@legalsylvain
[FIX] pos_user_restriction: disable test that are failing if pos_res…
0d591c4 
…taurant module is installed
@OCA-git-bot
pos_user_restriction 12.0.1.0.2
8bb0717
@hkapatel-initos
[IMP]pos_user_restriction: black, isort, prettier
9d20b3a
@hkapatel-initos
[MIG] pos_user_restriction: Migration to 14.0
28602c1
@AshishHirapara
[IMP][14.0] pos_user_restriction
37c9d0d 
Updated roadmap description
[IMP] pos_user_restriction: black, isort, prettier
8c89144
@ghost ghost marked this pull request as draft June 8, 2023 08:59
@ghost ghost marked this pull request as ready for review June 8, 2023 09:14
yankinmax
yankinmax approved these changes Jun 12, 2023
View reviewed changes
Copy link
Contributor

@yankinmax yankinmax left a comment

Choose a reason for hiding this comment

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

Did you check it functionally?

@ghost
Copy link
Author

ghost commented Jun 13, 2023

@yankinmax yes !

Camille0907
Camille0907 approved these changes Jun 19, 2023
View reviewed changes
Copy link

@Camille0907 Camille0907 left a comment

Choose a reason for hiding this comment

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

Technically looks good, thanks :)

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@ivantodorovich
Copy link
Contributor

/ocabot migration pos_user_restriction

@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jul 4, 2023
@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-1010-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Jul 4, 2023
Open
27 tasks
@OCA-git-bot OCA-git-bot merged commit 56caecf into OCA:15.0 Jul 4, 2023
@OCA-git-bot
Copy link
Contributor

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

@em230418 em230418 mentioned this pull request Oct 25, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.
9 participants
@OCA-git-bot @ivantodorovich @yankinmax @Camille0907 @eLBati @danimv5 @legalsylvain @hkapatel-initos @AshishHirapara