GSA / smartpay-training

Prototype for new GSA SmartPay training quizzes
8 stars 4 forks source link

Select Bureau during registration process #37

Closed JessicaMarine1 closed 1 year ago

JessicaMarine1 commented 1 year ago

For CFO Act agency users, there is a need for more granular reporting at the "bureau" level. To accommodate that need, during the "registration" process, users should select a bureau, as needed.

Acceptance Criteria:

Bureau of Alcohol, Tobacco, Firearms and Explosives (ATF) Drug Enforcement Administration (DEA) Federal Bureau of Prisons (BOP) Federal Bureau of Investigation (FBI) Federal Prison Industries (FPI) Justice Management Division-Offices, Boards and Divisions (OBD) Office of Justice Programs (OJP) U.S. Marshals Service (USMS) Other

Alcohol and Tobacco Tax and Trade Bureau Bureau of the Fiscal Service Bureau of Engraving and Printing Community Development Financial Institutions Fund Comptroller of the Currency Financial Crimes Enforcement Network Internal Revenue Service Special Inspector General for the Troubled Asset Relief Program Special Inspector General for Pandemic Recovery Treasury Inspector General for Tax Administration United States Mint Other

-United States Department of Agriculture

Agriculture Marketing Service Agriculture Research Service Animal and Plant Health Inspection Service Departmental Administration Economic Research Service Farm Production and Conservation Farm Service Agency Food Safety and Inspection Service Foreign Agricultural Service Forest Service National Agricultural Statistics Service National Appeals Division National Institute of Food and Agriculture Natural Resources Conservation Services Office of Budget and Program Analysis Office of Communications Office of Congressional Relations Office of External and Intergovernmental Affairs Office of Hearings and Appeals Office of Inspector General Office of Partnerships and Public Engagement Office of the Assistant Secretary for Civil Rights Office of the Chief Economist Office of the Chief Financial Officer Office of the Chief Information Officer Office of the Chief Scientist Office of the Executive Secretariat Office of Tribal Relations Risk Management Agency Rural Development The White House Liaison Office Other

JessicaMarine1 commented 1 year ago

@A-L33 I would recommend re-ordering the bureaus for each agency so that the bureaus are in the alphabetical order

A-L33 commented 1 year ago

Got it! Changed.

melchoyce commented 1 year ago

How's this? Note: Because combobox accessibility issues still aren't resolved in USWDS, let's just a regular select box for agency (as currently implemented) and subagency.

image Figma

A-L33 commented 1 year ago

Looks great!

A-L33 commented 1 year ago

Weird question... do we want an "Other" bureau to be a catch all? After thinking about it, I don't know if it's something that would be helpful. Might be something we want to test with the A/OPCs after we build it. I'm inclined to take out Other... thoughts?

melchoyce commented 1 year ago

@JessicaMarine1 and I chatted a bit about the combobox accessibility issues. Because the issues still aren't resolved in USWDS, let's just a regular select box instead for agency (so it remains the same as it is now) and subagency.

JessicaMarine1 commented 1 year ago

@A-L33 I think this is something that the program needs to decide. Since it will be a required field, are we sure all the options listed (if we exclude "other") will cover all users from a particular agency?

Weird question... do we want an "Other" bureau to be a catch all? After thinking about it, I don't know if it's something that would be helpful. Might be something we want to test with the A/OPCs after we build it. I'm inclined to take out Other... thoughts?

A-L33 commented 1 year ago

@rebekahperillo The more I think about this, the more I don't like "Other." I think either we get rid of it (with the implications Jessica mentioned) or we ask the A/OPCs if they want "Other."

rebekahperillo commented 1 year ago

@A-L33 I like "other" because I would think that the user would want to "check a box" if their bureau isn't an option. I'm not convinced that the user's bureau will ALWAYS be there....

A-L33 commented 1 year ago

@weiwang-gsa Bureau list is good to go for this story. I'll include any other agency/bureaus as a new story.

weiwang-gsa commented 1 year ago

@A-L33 question, previously in agency list, there is "Department of Agriculture", and here it is "United States Department of Agriculture", which one you want it to stand for agency name?

rebekahperillo commented 1 year ago

On their site, they say: U.S. Department of Agriculture. I would go with that.

Andrew, your thoughts?

Rebekah Knouse Perillo Business Management Specialist Center for Charge Card Management Office of Professional Services and Human Capital Categories Federal Acquisition Service, U.S. General Services Administration

301-938-5257 Follow the GSA SmartPay program on social media...Tweet with us on Twitter https://twitter.com/GSASmartPayNews and friend us on Facebook https://www.facebook.com/GSASmartPayNews/!

[image: image.png]

On Fri, Jun 2, 2023 at 12:24 PM Wei Wang @.***> wrote:

@A-L33 https://github.com/A-L33 question, previously in agency list, there is "Department of Agriculture", and here it is "United States Department of Agriculture", which one you want it to stand for agency name?

— Reply to this email directly, view it on GitHub https://github.com/GSA/smartpay-training/issues/37#issuecomment-1574006800, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7Z6HEXJ4IY6ERDINWV6FETXJIHTJANCNFSM6AAAAAAVMSAYNA . You are receiving this because you were mentioned.Message ID: @.***>

A-L33 commented 1 year ago

Let's use U.S. Department of Agriculture.

JessicaMarine1 commented 1 year ago

I'll add ^^^ to #209 -- i think we're going to have a few of these agency/bureau clean-up items before launch. Because each one would require a database migration, it's best to make them all at once.

A-L33 commented 1 year ago

According to our stand up conversation yesterday, I added #214 for additional agencies that are coming in. I just added HHS and Department of Commerce will submit by next Friday.

weiwang-gsa commented 1 year ago

@melchoyce this ticket is ready for design review: https://federalist-2e11f2c8-970f-44f5-acc8-b47ef6c741ad.sites.pages.cloud.gov/site/gsa/smartpay-training/

For the data part, I saw there is update in the bureau list ( there are others option added for last two), and maybe need update U.S. Department of Agriculture. That needs further discussion with engineers for best approach to do update scripts.

melchoyce commented 1 year ago

@weiwang-gsa Looks good to me!

weiwang-gsa commented 1 year ago

@A-L33 It is ready for QA: https://federalist-2e11f2c8-970f-44f5-acc8-b47ef6c741ad.sites.pages.cloud.gov/site/gsa/smartpay-training/

A-L33 commented 1 year ago

@weiwang-gsa Is it possible to change the order of the bureaus? They are listed in the acceptance criteria in alphabetical order. Or should we move this to #214?

weiwang-gsa commented 1 year ago

@A-L33 current code is order the list by name in ascending order, seems in postgressSQL "order by" by default doesn't take special character into consideration, it treat U.S. same as US , I am working on adding collation to it. should put all U.S. first then followed by US.

A-L33 commented 1 year ago

@weiwang-gsa For the Department of Transportation, the bureaus look like they might be in loose reverse order. Also, is there a way to anchor "other" to the bottom of the list?

Image

weiwang-gsa commented 1 year ago

@A-L33 ohh, I thought previously you are talking about Agencies order, For bureaus, you want the "Other" to be the last item in the bureau list? that is not by alphabetic order then, that will need to introduce new column in table like "sort_order", then it can support the specific order you want item to be displayed on UI. for that case, it will require database structure change and along with all the code update, then it will be better tracked in another story

A-L33 commented 1 year ago

@weiwang-gsa Understood. Is there an easy way for the bureaus to be listed alphabetically (we can leave "other" where it shows up on the list for now0? Or does alphabetizing the bureaus need to be tracked in another story too?

weiwang-gsa commented 1 year ago

@A-L33 the code change are made to main branch, bureaus are sorted by alphabetic order now. You may need to clear your cache to see the result change. https://federalist-2e11f2c8-970f-44f5-acc8-b47ef6c741ad.sites.pages.cloud.gov/site/gsa/smartpay-training/

A-L33 commented 1 year ago

Thank you! Done!