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

release(0.28.0): merge to prod #781

Merged
merged 9 commits into from
May 25, 2023
Merged

release(0.28.0): merge to prod #781

merged 9 commits into from
May 25, 2023

Conversation

seaerchin
Copy link
Contributor

What's Changed

Full Changelog: v0.27.0...v0.28.0

alexanderleegs and others added 9 commits May 19, 2023 09:31
* chore(axios): upgrade axios to 0.24 and add axios cache interceptor

pin axios to only allow minor patch upgrades, and pin the version of interceptor exactly. this is
because there is a support table
[here](https://axios-cache-interceptor.js.org/guide/getting-started) that lists the compat and we
require at least 0.6.x for ETag support. in order to prevent versioning errors (0.8.4 of interceptor
works wtih 0.25 of axios!!), we will pin the exact version of interceptor + patch version of axios

* feat(axiosinstance): add initialisation code

* chore(package): lock versions

* fix(axiosinstance): update config for cache interceptor
* feat(step functions service): add step functions service

* feat(dynamo cb client): add get all items function

* feat(config): add env var for step functions

* feat(dynamodb): integration with infra service

* test(dynamodb service): add test cases

* fix(infra service): fix wrong feature flag

* style(infra service): use env var rather than inline const

* feat(dynamodb service): add type guard

* fix(typeguard): fix type guard

* style(env var): change naming

* fix(typeguard): check for undefined values

* style(env var): better naming

* fix(config): give default value for step functions arn

* style(dynamoDb service): add args to constructor for ease of testing

* fix(infra service): add awaits

* fix(step functions): region should come from env vars

* fix(infra service): handle error appropriately
* feat: fetch only latest open PR from DB

* fix: ts error for null check

* fix: tests

* fix: rename route to listValidReviewRequests

* feat: filter for both open and approved PRs

* fix: tests
* fix(headerinterpreter): use custom interpreter

* build(ppackagge.json): add cache parser

* chore(headerinterpreter): prefix w/ custom

* fix(axiosinstance): preserve existing headers
* fix(directory): default to empty array

* chore(directory): remove console log
* refactor(parsepagename): add guard for markdownpage

* refactor(types/utils): update type def and shift util function otu

* feat(config): add parsing

* feat(pageservice): refactor to have stronger guarantee on method

* chore(errors): add new error types

* chore(configService): add precondition for pathinfo

* refactor(types): rejig

* refactor(rr service): more elaborate parsing n defaults

* ref(review): update call sites and add new method

* chore(services): init service

* fix(rr): change andThen to map

* chore(rr): ref to fiot new api

* chore(error): add status code

* fix(sites):  update typing and refactor for result

* chore(review): update review dto

* ref(types): add more github types

* chore(pages): add docs

* fix(tests): fixed tests and services

* chore(notifications spec): add missing imports

* test(tests): fixed all tests

* chore(types): update types and add default return
@seaerchin seaerchin changed the title release(0.28.0): merge to master release(0.28.0): merge to prod May 25, 2023
@mergify
Copy link

mergify bot commented May 25, 2023

⚠️ The sha of the head commit of this PR conflicts with #780. Mergify cannot evaluate rules on this PR. ⚠️

@seaerchin seaerchin requested a review from a team May 25, 2023 05:25
@seaerchin seaerchin merged commit 678a5ea into master May 25, 2023
@seaerchin seaerchin deleted the release/v0.28.0 branch May 25, 2023 08:03
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