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

[Admin][Product] Add view in store button #16268

Conversation

mpysiak
Copy link
Member

@mpysiak mpysiak commented May 17, 2024

Q A
Branch? bootstrap-admin-panel
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets n/a
License MIT

Screenshot 2024-05-17 at 07 33 52

Screenshot 2024-05-17 at 07 33 34

@mpysiak mpysiak requested review from a team as code owners May 17, 2024 05:01
@probot-autolabeler probot-autolabeler bot added the Admin AdminBundle related issues and PRs. label May 17, 2024
Copy link

github-actions bot commented May 17, 2024

Bunnyshell Preview Environment deployment failed

Check https://github.com/Sylius/Sylius/actions/runs/9127612679 for details.

Available commands:

  • /bns:deploy to redeploy the environment
  • /bns:delete to remove the environment

@mpysiak mpysiak force-pushed the SYL-3437-view-in-store-button branch from d218d82 to f7378ae Compare May 17, 2024 07:57
@NoResponseMate NoResponseMate merged commit 6d6079e into Sylius:bootstrap-admin-panel May 17, 2024
19 checks passed
@NoResponseMate
Copy link
Contributor

Thank you, @mpysiak!

GSadee added a commit that referenced this pull request May 20, 2024
…oResponseMate, GSadee, jakubtobiasz, Wojdylak, kulczy, Rafikooo, mpysiak)

This PR was merged into the 2.0 branch.

Discussion
----------

| Q               | A
|-----------------|-----
| Branch?         | 2.0
| Bug fix?        | no
| New feature?    |yes
| BC breaks?      | yes
| Deprecations?   | no
| Related tickets | 
| License         | MIT

As our new Bootstrap Admin Panel is already mostly covered and starting to be functional, I would like to merge branch `bootstrap-admin-panel` to `2.0` and continue the work here.

<!--
 - Bug fixes must be submitted against the 1.12 or 1.13 branches
 - Features and deprecations must be submitted against the 1.14 branch
 - Features, removing deprecations and BC breaks must be submitted against the 2.0 branch
 - Make sure that the correct base branch is set

 To be sure you are not breaking any Backward Compatibilities, check the documentation:
 https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->


Commits
-------

c046fb7 Use extension instead of changing the existing forms in ProductBundle
38c8be5 minor review fixes
115d4c2 Revoke change of AutocompleteChoiceType to AdminBundle's one
10a2cc5 move ProductType related extensions definitions to dedicated file
165b84c Bring back the transformer from ProductBundle and create a new one for Admin
eff1c8f Use ProductAssociationType instead of extension
2310f81 [Admin][DX] Remove autocompleters
480cfca [Admin][Product] Add ProductAttribute autocomplete type
9516ad0 [Admin][DX] Change `AutocompleteChoiceType` suffix to `AutocompleteType`
72411ad [Admin][DX] Add default aliases to autocomplete fields
6e256e6 [Admin][UX] Add more searchable fields to autocompletes
e694fbd Merge branch '2.0' into bootstrap-admin-panel
6b26f4a Add the "sylius.live_component" tag and the translation layer
04fc430 Move all components from the "TwigComponent" directory under the "Twig\Component" directory
c2f0596 Convert all AsLiveComponent attribute to the custom sylius.live_component tag
8823b74 Bump ux-symfony packages minimal version
32e2cca Update PHPStan ignore rules for Twig components
ce811f2 Fix the malformed sylius_admin:country:form component usages
2fd9297 Fix not adding the 'controller.service_arguments' tag to services tagged with sylius.live_component
1e30fb3 Bring back #[AsLiveComponent] attribute to LiveComponent classes
65969a7 Unfinalize all components
7be88ea Rename `TaxonTreeComponent` component to `TreeComponent`
7e52dfa Fix Product's Form component service id
e435ebf Rename `ShipComponent` to `ShipFormComponent`
ed65c29 Adjust the Zone's form component to the new standard
3d06429 [Behat] Change mink panther to chromedriver
bf3a434 Rename sylius_admin:order:history:address to sylius_admin:order:addresshistory
6f3c0d9 [Behat] Disable behat test
73b4a5f Unify index actions
4cc8549 Add show buttons to the homepage
064db37 Improve filter section
5e319d4 Make bolder header
d3748e9 Style grid fields templates
4e6f42d Improve grids view
7d37d78 Fix tests
5fd5746 [Admin][Zone] Move FormComponent
8b6d2b1 [Admin][Zone] Move FormComponent (#16256)
32fcf43 [Admin][Product][Inventory] Enable behat tests
08c3e85 [Admin][Taxon] Implements media management
630f9a1 [Admin][Taxon][Image] Enable behat tests
e6deeb4 [Behat] Submit live component before submitting the form
e3909ab [Behat] Extract waitForFormUpdate function to CrudFormElement
59695b1 [Taxon][Behat] Extract management of image to separate form element.
554da08 [Admin][Taxon] Remove unnecessary code
e30e391 [Admin][Taxon] Split media more granularly
f42496c [Admin][Taxon] Rename FormImageElement to ImageFormElement
36e06c0 Merge branch '2.0' into bootstrap-admin-panel
bb4e9f3 [Admin][UI] Add error indicators for form tabs and accordions
865a481 Review fixes
d1ed503 Remove final from the Zone's form component
e6bd06a Remove final from the Zone's form component (#16270)
cb9ac59 [Admin][UI] Add error indicators for form tabs and accordions (#16263)
27a9529 [Admin][Product] Add view in store button
d8e1758 [Admin][Product] Enable and fix tests
f7378ae Changes after CR
c940afe Merge branch '2.0' into bootstrap-admin-panel
a313b14 Fixes after CR
6d6079e minor #16268 [Admin][Product] Add view in store button (mpysiak)
68da42b [Maintenance] Enable non-UI tests in CI
71ff79c [Behat] Add possibility to accepting null as value in response checker
f083d82 [Admin][UX] Filter translatable autocomplete results based on current locale
@mpysiak mpysiak deleted the SYL-3437-view-in-store-button branch May 21, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants