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

Low-income use case #1872

Merged
merged 17 commits into from Feb 14, 2024
Merged

Low-income use case #1872

merged 17 commits into from Feb 14, 2024

Conversation

indexing
Copy link
Member

@indexing indexing commented Feb 3, 2024

No description provided.

@indexing indexing requested a review from a team as a code owner February 3, 2024 01:27
@github-actions github-actions bot added the documentation [auto] Improvements or additions to documentation label Feb 3, 2024
Copy link

github-actions bot commented Feb 3, 2024

Coverage report

This PR does not seem to contain any modification to coverable code.

Copy link

github-actions bot commented Feb 3, 2024

Preview url: https://benefits-1872--cal-itp-previews.netlify.app

1 similar comment
Copy link

github-actions bot commented Feb 3, 2024

@indexing
Copy link
Member Author

indexing commented Feb 3, 2024

I still need to add the sequence diagram for this use case, but I'll do that as a change after this file is merged.

Additionally, I'd like this to display alphabetically by name in the list of pathways. Is this a change I can make post-merge? Not sure how to do it before.

@thekaveman
Copy link
Member

@indexing the file is named Low-income.md which is why it is sorting above the all lowercase e.g. agency-card.md.

If you rename the file to be all lowercase like the rest, it will naturally sort.

@thekaveman
Copy link
Member

I fully agree with all of @machikoyasuda's comments above!

Copy link
Member

@angela-tran angela-tran left a comment

Choose a reason for hiding this comment

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

Thanks for all your work on this @indexing . I left one comment about a small typo.

Co-authored-by: machiko <machiko@compiler.la>
Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

indexing and others added 2 commits February 9, 2024 14:30
…response formatting

Co-authored-by: machiko <machiko@compiler.la>
Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

1 similar comment
Copy link

github-actions bot commented Feb 9, 2024

Co-authored-by: machiko <machiko@compiler.la>
Copy link

github-actions bot commented Feb 9, 2024

indexing and others added 2 commits February 9, 2024 14:48
Co-authored-by: machiko <machiko@compiler.la>
Co-authored-by: machiko <machiko@compiler.la>
Copy link

github-actions bot commented Feb 9, 2024

1 similar comment
Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

Copy link

github-actions bot commented Feb 9, 2024

1 similar comment
Copy link

github-actions bot commented Feb 9, 2024

Copy link


4. The transit rider authenticates with their existing [Login.gov](Login.gov) account or, if they don’t have one, creates a [Login.gov](Login.gov) account.

5. The Cal-ITP Benefits app interfaces with the [California Department of Transportation](https://dot.ca.gov/) Identity Gateway (IdG) to verify benefit eligibility. The IdG uses personal information shared by [Login.gov](Login.gov) to verify CalFresh participation status.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
5. The Cal-ITP Benefits app interfaces with the [California Department of Transportation](https://dot.ca.gov/) Identity Gateway (IdG) to verify benefit eligibility. The IdG uses personal information shared by [Login.gov](Login.gov) to verify CalFresh participation status.
5. The Cal-ITP Benefits app interfaces with the [California Department of Technology](https://cdt.ca.gov/) [Identity Gateway (IdG)](https://digitalidstrategy.cdt.ca.gov/primary-elements.html) to verify benefit eligibility. The IdG uses personal information shared by [Login.gov](Login.gov) to verify Calfresh participation status.

Copy link
Member

Choose a reason for hiding this comment

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

@indexing I found CDT's website for the identity gateway: https://digitalidstrategy.cdt.ca.gov/primary-elements.html so I'm adding it here

Copy link
Member

@machikoyasuda machikoyasuda left a comment

Choose a reason for hiding this comment

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

Everything looks good except for that last CDT/Caltrans link typo.

- increased linked text
- replaced CDT generic URL with a much more meaningful one
Copy link

@machikoyasuda machikoyasuda merged commit b1ecb05 into dev Feb 14, 2024
10 checks passed
@machikoyasuda machikoyasuda deleted the indexing-patch-2 branch February 14, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation [auto] Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants