forked from bigcommerce/cornerstone
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Releasing 5.5.0-rc.1 #13
Open
lhooker
wants to merge
348
commits into
harveynichols:master
Choose a base branch
from
bigcommerce:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bc-yevhenii-buliuk
force-pushed
the
master
branch
from
December 9, 2021 09:35
531dae7
to
0b25628
Compare
…ot reflect the requirements set in the store settings (#2117)
bc-yevhenii-buliuk
force-pushed
the
master
branch
from
March 9, 2022 11:12
5bf5375
to
5517ffe
Compare
…not read by screen reader correctly (#2441)
Both a leading and trailing quote are already present in the code that calls this template, so this trailing " breaks the HTML. If you look at the source in the demo theme you'll see this behavior https://cornerstone-light-demo.mybigcommerce.com/shop-all/garden/
…ountry-specific websites (step 1) (#2453) Co-authored-by: “bc-yevhenii-buliuk” <“yevhenii.buliuk@bigcommerce.com”>
Co-authored-by: sacr3dc0w <5056945+sacr3dc0w@users.noreply.github.com>
#2454) Co-authored-by: Vlad Dlogush <31504758+VladDlogush@users.noreply.github.com>
…t item (#2457) Co-authored-by: Vlad Dlogush <31504758+VladDlogush@users.noreply.github.com>
* account.js template literal fix * adding changelog * adding changelog * Update CHANGELOG.md
…#2462) * Add account-content wrapper so content doesn't flow outside page container * Update changelog * Remove newline --------- Co-authored-by: Volodymyr Krasnoshapka <88093058+BC-krasnoshapka@users.noreply.github.com>
fix: DATA-11867 Fix duplicating add_to_cart meta pixel events
Co-authored-by: sacr3dc0w <5056945+sacr3dc0w@users.noreply.github.com>
Bumps [braces](https://github.com/micromatch/braces) to 3.0.3 and updates ancestor dependencies [braces](https://github.com/micromatch/braces), [babel-jest](https://github.com/jestjs/jest/tree/HEAD/packages/babel-jest) and [grunt-cli](https://github.com/gruntjs/grunt-cli). These dependencies need to be updated together. Updates `braces` from 3.0.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](micromatch/braces@3.0.2...3.0.3) Updates `babel-jest` from 25.5.1 to 29.7.0 - [Release notes](https://github.com/jestjs/jest/releases) - [Changelog](https://github.com/jestjs/jest/blob/main/CHANGELOG.md) - [Commits](https://github.com/jestjs/jest/commits/v29.7.0/packages/babel-jest) Updates `grunt-cli` from 1.3.2 to 1.4.3 - [Release notes](https://github.com/gruntjs/grunt-cli/releases) - [Changelog](https://github.com/gruntjs/grunt-cli/blob/main/CHANGELOG.md) - [Commits](gruntjs/grunt-cli@v1.3.2...v1.4.3) --- updated-dependencies: - dependency-name: braces dependency-type: indirect - dependency-name: babel-jest dependency-type: direct:development - dependency-name: grunt-cli dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update mobile-menu-toggle.js Adds focus trapping for mobile nav dropdown * Adding focusTrap import * mobile focustrap update * remove related code
* Load other font weights and styles for the body-font Using a font name like `Google_Karla_400` for `body-font`, it would only load the 400 font weight. When using `Google_Karla_700` for another property, it would then load both the 400 and 700 fon weights. But since in CSS you could make something bold or italic, the font weight and/or style for this text might be missing, as it was not loaded automatically by using it in another property. This PR will make sure that the most common font weights and font styles will be loaded for the body font. * Update CHANGELOG.md --------- Co-authored-by: Volodymyr Krasnoshapka <88093058+BC-krasnoshapka@users.noreply.github.com>
…30th and 31st (#2473) Co-authored-by: Vlad Dlogush <31504758+VladDlogush@users.noreply.github.com>
Co-authored-by: Vlad Dlogush <31504758+VladDlogush@users.noreply.github.com>
…mer account displays incorrect quantity of products ordered compared to Order details page (#2482) Co-authored-by: Vlad Dlogush <31504758+VladDlogush@users.noreply.github.com>
…ax price lists on PDP (#2486)
fix(storefront): BCTHEME-1985 Fix stored XSS within company address field
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
A description about what this pull request implements and its purpose. Try to be detailed and describe any technical details to simplify the job of the reviewer and the individual on production support.
Tickets / Documentation
Add links to any relevant tickets and documentation.
Screenshots (if appropriate)
Attach images or add image links here.