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

Connect to API Gateway #153

Merged
merged 37 commits into from
Jun 7, 2021
Merged

Connect to API Gateway #153

merged 37 commits into from
Jun 7, 2021

Conversation

kevmo
Copy link
Contributor

@kevmo kevmo commented May 11, 2021

This PR represents our work connecting with the API gateway in a secure way. Some code/comments are there to mark places that need work once development from the API team is finished (e.g. the unique number header we are expecting from UIO, an API response with end user information in it rather than a placeholder).


Environmental variables (in a root-level .env file) are used to make the code more configurable in the event of URL/header/parameter changes in the future.

===

Resolves #XXX

  • Documentation (e.g. READMEs) updated, if relevant

@kevmo kevmo added Engineering Technology and Development Track WIP labels May 11, 2021
@kevmo kevmo self-assigned this May 11, 2021
@kevmo kevmo requested review from kalvinwang and lomky as code owners May 11, 2021 00:02
package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@kalvinwang kalvinwang left a comment

Choose a reason for hiding this comment

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

Thanks @kevmo! Added some comments and questions.

.gitignore Outdated Show resolved Hide resolved
components/ClaimCard.tsx Outdated Show resolved Hide resolved
components/ClaimCard.tsx Outdated Show resolved Hide resolved
pages/_app.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
pages/claimstatus/index.tsx Outdated Show resolved Hide resolved
@lomky lomky mentioned this pull request May 20, 2021
3 tasks
@rocketnova rocketnova self-assigned this Jun 1, 2021
Copy link
Contributor

@rocketnova rocketnova left a comment

Choose a reason for hiding this comment

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

@kalvinwang I think this is ready for review

README.md Show resolved Hide resolved
pages/index.tsx Show resolved Hide resolved
@rocketnova rocketnova changed the title WIP: Connecting to the API Gateway. Connect to API Gateway Jun 2, 2021
@rocketnova rocketnova requested a review from kalvinwang June 3, 2021 00:09
@rocketnova rocketnova removed the WIP label Jun 3, 2021
Copy link
Contributor

@kalvinwang kalvinwang left a comment

Choose a reason for hiding this comment

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

lgtm! Just some minor edits and questions about making tickets from todos. Also maybe add one more ticket for cleaning up types in index.tsx?

.eslintrc.js Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
pages/index.tsx Outdated Show resolved Hide resolved
pages/index.tsx Outdated Show resolved Hide resolved
pages/index.tsx Outdated Show resolved Hide resolved
pages/index.tsx Outdated Show resolved Hide resolved
rocketnova and others added 3 commits June 7, 2021 11:57
@rocketnova rocketnova merged commit 8cb9f23 into main Jun 7, 2021
@rocketnova rocketnova deleted the kevmo/api-gateway branch June 7, 2021 19:03

import { Header } from '../components/Header'
import { Main } from '../components/Main'
import { Footer } from '../components/Footer'

export default function Home(): ReactElement {
export interface Claim {
ClaimType: string | 'not working'
Copy link
Contributor

@kalvinwang kalvinwang Jun 7, 2021

Choose a reason for hiding this comment

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

should this be ?? '' for now and error handled later?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Technology and Development Track
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants