Skip to content
This repository was archived by the owner on Dec 6, 2024. It is now read-only.

Commit b5e543e

Browse files
trietluchasechow7mabreuortegaMiguel Abreumiguelpdiaz8
authored
merged changes for 1.1.3 from mainline-dev onto mainline (#672)
* Add accessibility label to checkboxes in Check Data (#648) Added accessibility labels to the checkboxes so the text-to-speech screen reader will output the column header name * Miguel.abreu/gtt 1583 y axis labels cut off (#646) * Refactor bar char size logic * Added RulerService to measure how many pixels a word takes in the screen * Refactor yAxis formatter to trim the label at a size that fits in one line preventing the fault logic of label splitter provided by recharts * RulerService unit tests * Minor refactor for ruler div creation * Encode ruler div content Co-authored-by: Miguel Abreu <miabreu@amazon.com> * Fix position of actions dropdown on mobile (#641) * Store favicon image name (#649) Co-authored-by: Miguel Pavon Diaz <71112226+miguelpdiaz8@users.noreply.github.com> * Fix section highlight while scrolling up (#650) * Add new alt text field (#656) Co-authored-by: Miguel Abreu <miabreu@amazon.com> * logo name (#655) Co-authored-by: Miguel Pavon Diaz <71112226+miguelpdiaz8@users.noreply.github.com> * Add reactive error handling and RadioButtonTile component on Choose data step (#657) * Enforce test coverage thresholds (#659) Backend: 70% coverage Frontend: 45% coverage Co-authored-by: Miguel Abreu <miabreu@amazon.com> * Update README info (#660) * Use container width for computation (#661) Co-authored-by: Miguel Abreu <miabreu@amazon.com> * Click scrolls when not visible (#662) * Manipulating DOM to click scrolls when not visible * Manipulating DOM to click scrolls when not visible * Move logic to the Tabs container * Move to react apis * Remove unused import * Remove custom logs Co-authored-by: Miguel Abreu <miabreu@amazon.com> * GTT-1590 - Removed 403 error handler. Added bucket policy to allow for CF to access bucket (#663) * Remove 403 error handler. Add bucket policy to allow for CF to access bucket * enabled prettier check * Ran prettier on WAF template * GTT 1602 : bump ansi-regex from 5.0.0 to 5.0.1 in backend (#667) * bump ansi-regex from 5.0.0 to 5.0.1 * Trigger Build * bump version of ansi-regex per dependabot finding * bump ansi-regex * Upgrade to 1.1.3 (#670) * Update runbook (#668) Co-authored-by: Triet <triet05@gmail.com> * New translated labels (#671) Co-authored-by: chasechow7 <92477064+chasechow7@users.noreply.github.com> Co-authored-by: Miguel Alfonso Abreu Ortega <mabreuortega@gmail.com> Co-authored-by: Miguel Abreu <miabreu@amazon.com> Co-authored-by: Miguel Pavon Diaz <71112226+miguelpdiaz8@users.noreply.github.com>
1 parent 2b3244c commit b5e543e

File tree

76 files changed

+1706
-789
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

76 files changed

+1706
-789
lines changed

CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

8+
## [1.1.3] - 2021-10-28
9+
10+
### Added
11+
12+
- On public dashboard page, scrolling up now highlights a Section's table of contents entry
13+
- Improved accessiblity by adding labels to the Check Data column checkboxes and alt-text for the logo upload in Settings
14+
- Uploaded favicon retains original filename
15+
816
## [1.1.2] - 2021-10-14
917

1018
### Added

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ PDoA comes with pre-built code to provision an instance in your AWS account. You
3333

3434
| Region | Launch |
3535
| -------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
36-
| Install in us-east-1 | [![Install in us-east-1](docs/images/launch-stack.svg)](https://console.aws.amazon.com/cloudformation/home?region=us-east-1#/stacks/quickcreate?templateURL=https://performance-dashboard-on-aws-solution-releases-us-west-2.s3.us-west-2.amazonaws.com/performance-dashboard-on-aws/v1.1.2/performance-dashboard-on-aws.template) |
36+
| Install in us-east-1 | [![Install in us-east-1](docs/images/launch-stack.svg)](https://console.aws.amazon.com/cloudformation/home?region=us-east-1#/stacks/quickcreate?templateURL=https://performance-dashboard-on-aws-solution-releases-us-west-2.s3.us-west-2.amazonaws.com/performance-dashboard-on-aws/v1.1.3/performance-dashboard-on-aws.template) |
3737

3838
### Clone this repository
3939

backend/jest.config.js

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
11
module.exports = {
2-
preset: 'ts-jest',
3-
testEnvironment: "node",
4-
roots: [
5-
"<rootDir>/src"
6-
],
7-
}
2+
preset: "ts-jest",
3+
testEnvironment: "node",
4+
roots: ["<rootDir>/src"],
5+
coverageThreshold: {
6+
global: {
7+
branches: 70,
8+
functions: 80,
9+
lines: 80,
10+
statements: 80,
11+
},
12+
},
13+
};

backend/package-lock.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

backend/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "performance-dashboard-backend",
3-
"version": "1.1.2",
3+
"version": "1.1.3",
44
"description": "Performance Dashboard on AWS Backend",
55
"awssdkUserAgent": "AwsSolution/SO0157/v",
66
"scripts": {

backend/src/lib/controllers/settings-ctrl.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ async function updateSettings(req: Request, res: Response) {
2020
navbarTitle,
2121
topicAreaLabels,
2222
customLogoS3Key,
23+
customLogoAltText,
2324
customFaviconS3Key,
2425
colors,
2526
adminContactEmailAddress,
@@ -90,6 +91,15 @@ async function updateSettings(req: Request, res: Response) {
9091
);
9192
}
9293

94+
if (customLogoAltText) {
95+
updatedAt = await repo.updateSetting(
96+
"customLogoAltText",
97+
customLogoAltText,
98+
updatedAt,
99+
user
100+
);
101+
}
102+
93103
if (customFaviconS3Key) {
94104
updatedAt = await repo.updateSetting(
95105
"customFaviconS3Key",

backend/src/lib/factories/__tests__/settings-factory.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ describe("fromItem", () => {
4343
plural: "Topic Areas",
4444
},
4545
customLogoS3Key: "12345",
46+
customLogoAltText: "Alt text",
4647
customFaviconS3Key: "12345",
4748
contactEmailAddress: "test@aol.com",
4849
adminContactEmailAddress: "admin@aol.com",
@@ -62,6 +63,7 @@ describe("fromItem", () => {
6263
plural: "Topic Areas",
6364
},
6465
customLogoS3Key: "12345",
66+
customLogoAltText: "Alt text",
6567
customFaviconS3Key: "12345",
6668
colors: {
6769
primary: "#2491ff",
@@ -88,6 +90,7 @@ describe("toPublicSettings", () => {
8890
plural: "Topic Areas",
8991
},
9092
customLogoS3Key: "12345",
93+
customLogoAltText: "Alt text",
9194
customFaviconS3Key: "12345",
9295
colors: {
9396
primary: "#ffffff",

backend/src/lib/factories/settings-factory.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ function getDefaultSettings(): Settings {
1616
plural: "Topic Areas",
1717
},
1818
customLogoS3Key: undefined,
19+
customLogoAltText: undefined,
1920
customFaviconS3Key: undefined,
2021
colors: {
2122
primary: "#2491ff",
@@ -41,6 +42,9 @@ function fromItem(item: SettingsItem): Settings {
4142
customLogoS3Key: item.customLogoS3Key
4243
? item.customLogoS3Key
4344
: defaults.customLogoS3Key,
45+
customLogoAltText: item.customLogoAltText
46+
? item.customLogoAltText
47+
: defaults.customLogoAltText,
4448
customFaviconS3Key: item.customFaviconS3Key
4549
? item.customFaviconS3Key
4650
: defaults.customFaviconS3Key,

backend/src/lib/models/settings.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ export interface Settings {
1111
plural: string;
1212
};
1313
customLogoS3Key: string | undefined;
14+
customLogoAltText: string | undefined;
1415
customFaviconS3Key: string | undefined;
1516
colors?: {
1617
primary: string;
@@ -55,6 +56,7 @@ export interface SettingsItem {
5556
plural: string;
5657
};
5758
customLogoS3Key: string | undefined;
59+
customLogoAltText: string | undefined;
5860
customFaviconS3Key: string | undefined;
5961
colors?: {
6062
primary: string;

backend/src/lib/repositories/__tests__/settings-repo.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ describe("getSettings", () => {
5252
plural: "Topic Areas",
5353
},
5454
customLogoS3Key: "12345",
55+
customLogoAltText: "Alt text",
5556
customFaviconS3Key: "12345",
5657
};
5758

0 commit comments

Comments
 (0)