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

#996 add url fragment to pager #1031

Merged
merged 1 commit into from
Feb 12, 2015
Merged

#996 add url fragment to pager #1031

merged 1 commit into from
Feb 12, 2015

Conversation

tzyganu
Copy link
Contributor

@tzyganu tzyganu commented Feb 9, 2015

I added support for url fragment in the pager as discussed in #996 .
And while at it, I made the method setAvailableLimit return $this. Originally it did not return anything and this made it unavailable for chaining.

@antonkril
Copy link
Contributor

Method chaining is considered harmful :)

@tzyganu
Copy link
Contributor Author

tzyganu commented Feb 9, 2015

@antonkril Then you can remove the return $this from the setAvailableLimit method when you remove the return $this lines from all the other one million places it is used in the core code 😄

@vpelipenko vpelipenko self-assigned this Feb 10, 2015
@vpelipenko
Copy link
Contributor

CR: passed
Builds: green
Resolution: OK to merge
Ticket: MAGETWO-34066

vpelipenko added a commit that referenced this pull request Feb 12, 2015
@vpelipenko vpelipenko merged commit 97b742d into magento:develop Feb 12, 2015
mmansoor-magento pushed a commit to magento-mpi/magento2 that referenced this pull request May 9, 2016
magento-team pushed a commit that referenced this pull request Apr 20, 2017
[Falcons] Delivery of deployment improvements
magento-engcom-team added a commit that referenced this pull request Mar 24, 2018
…mage instead variation image #1031

 - Merge Pull Request magento-engcom/magento2ce#1031 from RomaKis/magento2ce:8168
 - Merged commits:
   1. 93a5c39
magento-engcom-team pushed a commit that referenced this pull request Mar 24, 2018
fe-lix- pushed a commit to fe-lix-/magento2 that referenced this pull request Apr 29, 2018
magento-engcom-team added a commit that referenced this pull request Nov 14, 2019
…Mutation createCustomerAddress #1031

 - Merge Pull Request magento/graphql-ce#1031 from osrecio/graphql-ce:1029-postcode-required-customerCreate
 - Merged commits:
   1. 676cd1b
   2. 127835e
   3. 891e5a4
magento-engcom-team pushed a commit that referenced this pull request Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants