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.

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

9.0 auth admin passkey #522

Closed
wants to merge 2 commits into from
Closed

9.0 auth admin passkey #522

wants to merge 2 commits into from

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Aug 18, 2016

same as 8.0 #519
but also migrated the view and enable the module on 9.0
cc @legalsylvain @pedrobaeza

@pedrobaeza
Copy link
Member

Please change module version to 9.0.1.0.0.

This was referenced Aug 18, 2016
Closed
Closed
@pedrobaeza
Copy link
Member

See the comments in 8.0 version.

@pedrobaeza pedrobaeza mentioned this pull request Aug 23, 2016
Closed
lasley
lasley reviewed Aug 23, 2016
View reviewed changes
auth_admin_passkey/__openerp__.py
@@ -41,6 +24,6 @@
'images': [],
'post_load': '',
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove empty keys

@rvalyi
Copy link
Member Author

rvalyi commented Aug 23, 2016 •
edited
Loading

@lasley @JayVora-SerpentCS thanks for the review. Guys I'll apply these changes on #519 for the remaining ones and then cherry-pick and amend the commit from 8.0 here. cc @legalsylvain . Once I started making changes in the 8.0 PR I put the "work in progress" label on this 9.0 PR, I'm sorry if that wasn't clear.

@pedrobaeza
Copy link
Member

I have merged the other PR. Please include the changes and attend comments here.

@nicomacr nicomacr mentioned this pull request Dec 29, 2016
Closed
@pedrobaeza pedrobaeza mentioned this pull request Jan 25, 2017
Closed
@JonathanNEMRY
Copy link

I think this branche should be "rebase" to be complient with the last commit 601109e . Moreover, what's the current status of this one?

JonathanNEMRY pushed a commit to acsone/server-tools that referenced this pull request Feb 23, 2017
[CHG] review of OCA#522 & add setup for auth_admin
5802b96
JonathanNEMRY pushed a commit to acsone/server-tools that referenced this pull request Feb 23, 2017
[CHG] review of OCA#522 & add setup for auth_admin
4ea9b7a
JonathanNEMRY pushed a commit to acsone/server-tools that referenced this pull request Feb 23, 2017
[CHG] review of OCA#522 & add setup for auth_admin
cc5a8b9
@JonathanNEMRY
Copy link

I finally applied changes into akretion#14 & akretion#15 Once @rvalyi merged 15 and 14 I think this one will be ok to be merged into oca/9.0

@JonathanNEMRY JonathanNEMRY mentioned this pull request Mar 29, 2017
Merged
@pedrobaeza
Copy link
Member

@rvalyi this one is conflicting. Are you going to finish this?

@github-actions
Copy link

github-actions bot commented Feb 6, 2022

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 6, 2022
@rvalyi rvalyi closed this Feb 10, 2022
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#522 from OCA/12.0
2cbd602 
Syncing from upstream OCA/server-tools (12.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JonathanNEMRY JonathanNEMRY JonathanNEMRY left review comments

Assignees
No one assigned
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically. work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rvalyi @pedrobaeza @JonathanNEMRY @JayVora-SerpentCS @lasley