Skip to content

Commit

Permalink
Merge pull request #18 from adjust/v121
Browse files Browse the repository at this point in the history
Version 1.2.1
  • Loading branch information
YaraMatkova authored Jul 31, 2024
2 parents 12e4db0 + 95ece77 commit b55ac85
Show file tree
Hide file tree
Showing 8 changed files with 33 additions and 22 deletions.
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
### Version 1.2.1 (1st August 2024)

#### Fixed
- Fixed an issue where calling `show` immediately after `init` when a network error occurred resulted in an infinite loop.
- Added aria-label for dismiss button.

---

### Version 1.2.0 (10th June 2024)

#### Added
Expand Down
4 changes: 2 additions & 2 deletions demo/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@
"test": "echo \"Error: no test specified\" && exit 1"
},
"dependencies": {
"@adjustcom/smart-banner-sdk": "^1.2.0",
"@adjustcom/smart-banner-sdk-layout": "^3.1.0",
"@adjustcom/smart-banner-sdk": "^1.2.1",
"@adjustcom/smart-banner-sdk-layout": "^3.1.1",
"lorem-ipsum": "^2.0.8"
},
"author": "Adjust GmbH",
Expand Down
2 changes: 1 addition & 1 deletion layout/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@adjustcom/smart-banner-sdk-layout",
"version": "3.1.0",
"version": "3.1.1",
"description": "Adjust Smart Banner SDK Layout",
"scripts": {
"build": "npm run build:src && npm run build:types",
Expand Down
1 change: 1 addition & 0 deletions layout/src/dismiss-button/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ export class DismissButton {
this.button = document.createElement('button');
this.button.innerHTML = cross;
this.button.className = styles.dismiss;
this.button.ariaLabel = 'Close banner';

const svg = this.button.querySelector('svg');
if (svg) {
Expand Down
32 changes: 16 additions & 16 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@adjustcom/smart-banner-sdk-project",
"version": "1.2.0",
"version": "1.2.1",
"description": "Adjust Smart Banner SDK",
"scripts": {
"build": "npm run build --workspace=layout && npm run build --workspace=sdk && npm run build:snippet && npm run copy",
Expand Down
2 changes: 1 addition & 1 deletion sdk/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@adjustcom/smart-banner-sdk",
"version": "1.2.0",
"version": "1.2.1",
"description": "Adjust Smart Banner SDK",
"scripts": {
"build:sdk": "webpack --config ./webpack.config.js",
Expand Down
4 changes: 3 additions & 1 deletion sdk/src/domain/banner-provider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,11 @@ export class BannerProvider {
}

this.selectedBanner = this.selector.next(bannersList, url);
this.gettingBannerPromise = null;

return this.selectedBanner;
})
.finally(() => {
this.gettingBannerPromise = null;
});

return this.gettingBannerPromise;
Expand Down

0 comments on commit b55ac85

Please sign in to comment.