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

SmartPay Website Release changes v2.0 #380

Merged
merged 159 commits into from
Dec 1, 2023
Merged

Conversation

felder101
Copy link
Contributor

Includes (Sprint 19 and 20 stories):

Issue 370 USWDS update from 3.5 to 3.6.1
Issue 373 FASCSA Exclusion Link and Language
Content changes

mark-meyer and others added 30 commits October 11, 2023 09:42
Documentation and Code Review
Updating intro text to meet USWDS suggestion.
Update intro text to meet USWDS guidance.
Shorten intro to align with USWDS guidance.
Update card/account holder language.
Updated goods/services language.
Update intro text.
Update/streamline content.
Add an intro.
Add intro to this page.
Update Program to program per Style Guide.
Updated intro and card/account language per the Style Guide.
Updating the intro and card/account references per the Style Guide.
Update intro.
Update intro.
Intro update.
Added an intro.
Update intro text.
Update content flow.
Update card/account per Style Guide.
Updated to reflect card/account holder per the Style Guide.
Updated intro.
Update intro.
rebekahperillo and others added 18 commits November 29, 2023 10:49
Update intro.
Update content to match with the Style Guide.
Updated employee information.
Update formatting issue.
Update integrated information.
Updated integrated information.
Updated IBAs to IBA.
Updated formatting issue.
Updating format issue.
Includes (Sprint 19 and 20 stories):

Issue 370 USWDS update from 3.5 to 3.6.1
Issue 373 FASCSA Exclusion Link and Language
Content changes
Copy link
Contributor

@mark-meyer mark-meyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, this looks great! There are a few comments below for some minor issues — let me know if you have any questions.

@@ -1,6 +1,6 @@
---
title: About GSA SmartPay®
description: "The GSA SmartPay program is the world’s largest government charge card and commercial payment solutions program."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think deleting the description in this metadata will result in an empty <meta name="description"> tag on the /about/ page. This looks like it's not intentional.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The description has been added back. Removed by mistake.

src/content/faq/faq.mdx Show resolved Hide resolved
The federal government issued an [interim rule](https://www.federalregister.gov/documents/2023/10/05/2023-21320/federal-acquisition-regulation-implementation-of-federal-acquisition-supply-chain-security-act) that amends the Federal Acquisition Regulation (FAR) to implement supply chain risk information sharing and FASCSA orders. This rule became effective on December 4, 2023.

## Accessing FASCSA Orders
[The System for Award Management (SAM)](http://sam.gov) is an official website of the U.S. Government that helps users navigate the federal award lifecycle.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is minor and doesn't have much effect on end users, but I wonder if we should make the link to https://.... to avoid making the browser redirect from http to https (same comment of the other link below).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated links to include https instead of http.

- [Sales, Transactions, and Account Holder Data (FY 2009–September 2022) [XLSX, 7MB]](/files/stats-tool.xlsx) – Spreadsheets containing current statistics for GSA SmartPay Purchase, Travel and Fleet. Note: For questions about usage of the tool and underlying data, refer to the [Frequently Asked Questions](/faq) page.
- [Fiscal Years 1999–2009 Card Sales, Transactions and Account Holder Data [XLS, 10.2 MB]](/files/fy99-09-card-sales-transactions-cardholder-data.xls) – Spreadsheets containing fiscal years 1999–2009 data and statistics for GSA SmartPay Purchase, Travel and Fleet.
- [Sales, Transactions, and Card/Account Holder Data (FY 2009–July 2023) [XLSX, 8MB]](/files/stats-tool.xlsx) – Spreadsheets containing current statistics for GSA SmartPay Purchase, Travel and Fleet business lines. Note: For questions about usage of the tool and underlying data, refer to the [Frequently Asked Questions](/faq) page.
- [Fiscal Years 1999–2009 Card Sales, Transactions and Card/Account Holder Data [XLS, 10.2 MB]](/files/fy99-09-card-sales-transactions-cardholder-data.xls) – Spreadsheets containing fiscal years 1999–2009 data and statistics for GSA SmartPay Purchase, Travel and Fleet business lines.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A little outside my swim lane, but it looks like other place on the site are using the Oxford comma. Should this be GSA SmartPay Purchase, Travel, and Fleet business lines. with the comma after Travel?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch! Added missing commas in the file.

Copy link
Contributor

@mark-meyer mark-meyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! This looks good to me.

@felder101 felder101 merged commit 7eb6e03 into main Dec 1, 2023
5 checks passed
@felder101 felder101 deleted the GSA/staging/ProdRelease branch December 1, 2023 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants