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

Remove authorized application entity and related controller and tests #34146

Merged
merged 6 commits into from Oct 3, 2023

Conversation

jolelievre
Copy link
Contributor

@jolelievre jolelievre commented Oct 3, 2023

Questions Answers
Branch? develop
Description? Remove authorized application entity and related controller and tests
Type? refacto
Category? BO
BC breaks? no
Deprecations? no
How to test? CI green and UI tests green
UI Tests https://github.com/jolelievre/ga.tests.ui.pr/actions/runs/6393313823
Fixed issue or discussion? Fixes #33721
Related PRs ~
Sponsor company ~

@jolelievre jolelievre requested review from a team as code owners October 3, 2023 12:37
@prestonBot prestonBot added develop Branch Refactoring Type: Refactoring labels Oct 3, 2023
@jolelievre jolelievre changed the title Remove authorize application entity and related controller and tests Remove authorized application entity and related controller and tests Oct 3, 2023
tleon
tleon previously approved these changes Oct 3, 2023
Copy link
Contributor

@nicosomb nicosomb left a comment

Choose a reason for hiding this comment

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

-3559
+59

❤️

@jolelievre jolelievre merged commit d75f4fc into PrestaShop:develop Oct 3, 2023
24 checks passed
@jolelievre jolelievre deleted the clean-applications branch October 3, 2023 14:18
@matks
Copy link
Contributor

matks commented Oct 9, 2023

@jolelievre Don't forget the milestone 😉

@matks matks added this to the 9.0.0 milestone Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
develop Branch Refactoring Type: Refactoring
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Delete Authorization seveur management pages
6 participants