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

Allow camelcase in vendorname for menus #985

Merged
merged 2 commits into from
Feb 13, 2015
Merged

Allow camelcase in vendorname for menus #985

merged 2 commits into from
Feb 13, 2015

Conversation

athopen
Copy link

@athopen athopen commented Jan 21, 2015

Currently you are not able to use camelcase in your vendorname if you create backend-menus.

andreaspenz added 2 commits January 21, 2015 08:55
- changed regex in menu.xsd
- fix files for unit-tests
@avoelkl
Copy link
Contributor

avoelkl commented Jan 21, 2015

+1

@rrohaczek
Copy link

👍

@marcelmacasso
Copy link

+1

@ateufel
Copy link

ateufel commented Jan 21, 2015

👍

@pixelhed
Copy link
Contributor

+1

1 similar comment
@ihor-sviziev
Copy link
Contributor

👍

@mzeis
Copy link
Contributor

mzeis commented Jan 26, 2015

Nice idea!

@otoolec
Copy link
Contributor

otoolec commented Jan 28, 2015

Thank you for this contribution! I've created ticket MAGETWO-33373 internally to process this pull request.

@otoolec otoolec added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development bug report and removed improvement labels Feb 3, 2015
@otoolec otoolec self-assigned this Feb 10, 2015
@vpelipenko
Copy link
Contributor

CR: passed
Builds: green
Resolution: OK to merge

vpelipenko added a commit that referenced this pull request Feb 13, 2015
Allow camelcase in vendorname for menus
@vpelipenko vpelipenko merged commit 607c343 into magento:develop Feb 13, 2015
otoolec added a commit that referenced this pull request Feb 13, 2015
Modify valid menu xml file test to catch a regression of this fix.
magento-team added a commit to abeeskau/magento2-community-edition that referenced this pull request Feb 20, 2015
* Various improvements:
    * Existing Builders were replaced with DataFactories in Customer and Tax modules
    * Refactored controller actions in the Checkout and CMS modules
    * Increased coverage with static tests for `.phtml` files
    * Moved Cookie related functionality from `Theme` and `Core` modules into a new `Cookie` module
    * Moved minfication configuration settings to the `View` library level
* UI improvements:
    * Restyled installation wizard
    * Prepared styles for Dashboard in the Backend area
* Framework improvements:
    * Added `setCustomAttribute` and `setCustomAttributes` methods to `ExtensibleDataInterface`
    * Added setter methods to data object interfaces
    * Replaced `Builders` with `Factories`
    * Added `DataObjectHelper.php` which contains the common set of methods of all builders
    * Refactored `__()` to return `Phrase` object
    * Allowed usage of `LocalizedException` on the framework's library level
    * Added expiration/lifetime management of frontend resources
    * Unified MTF configurations format for Framework, TestCase variations and TestCase scenario configurations
* Fixed bugs:
    * Fixed an issue with product reviews list paging
    * Fixed an issue where sold products were not displayed in Bestsellers
    * Fixed an issue with image rendering on the CMS page on Frontend when `webserver rewrites = no`
* GitHub requests:
    * [#790](magento/magento2#790) -- Magento API fails in a CGI env (zf1 issue)
    * [#909](magento/magento2#909) -- Manage Titles in popup window front-end issue
    * [#996](magento/magento2#996) -- Pager block should support url "fragment".
    * [#985](magento/magento2#985) -- Allow camelcase in vendorname for menus
    * [#1025](magento/magento2#1025) -- Wrong parameter for getting base url for 'media' path in "Image" form element.
magento-team pushed a commit that referenced this pull request Dec 11, 2017
…rk size Validation Message. #985

 - Merge Pull Request magento-engcom/magento2ce#985 from nmalevanec/magento2:12613
 - Merged commits:
   1. 74dbd7d
magento-team pushed a commit that referenced this pull request Dec 11, 2017
magento-team pushed a commit that referenced this pull request Dec 11, 2017
[EngCom] Public Pull Requests - 2.2-develop
 - MAGETWO-85311: Added namespace to product videos fotorama events #12469 #991
 - MAGETWO-85300: 8437: Silent error when an email template is not found #970
 - MAGETWO-85293: 12613: Verbiage Update Required: Product Image Watermark size Validation Message. #985
 - MAGETWO-85286: 8176: LinkManagement::getChildren() does not include product visibility. #986
 - MAGETWO-85285: 12482: Sitemap image links in MultiStore #935
 - MAGETWO-84955: Set Current Store from Store Code if isUseStoreInUrl #12529
 - MAGETWO-84764: NewRelic: Disables Module Deployments, Creates new Deploy Marker Command #12477
 - MAGETWO-84439: 12180 Remove unnecessary use operator for Context, causes 503 error i… #12220
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Progress: accept
Projects
None yet
Development

Successfully merging this pull request may close these issues.