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.

Sign up for GitHub

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

Jump to bottom

Production deploy 2/29 #239

Merged
merged 64 commits into from
Feb 29, 2024
Merged

Production deploy 2/29 #239

merged 64 commits into from
Feb 29, 2024

Conversation

levinmr
Copy link
Contributor

@levinmr levinmr commented Feb 29, 2024

No description provided.

scottqueen-bixal and others added 30 commits December 7, 2023 11:30
@scottqueen-bixal
lock in with node version 18
27b6f57
@scottqueen-bixal
propose env values for local developement
cf19987
@scottqueen-bixal
update package to version 1.2
93bbd82
@scottqueen-bixal
create new table for ga4 data
411553e
@scottqueen-bixal
handle versioning semantically
ad86701
@scottqueen-bixal
return table based on version
58f4acf
@scottqueen-bixal
remove console.log
9fb01a8
@XavierMetichecchia
Merge pull request #214 from 18F/migrate-ga3-to-ga4
5c8e000 
Migrate ga3 to ga4
Remove pipeline test
825a510
@scottqueen-bixal
Update api-data-gov-filter.js
13f24ed
@XavierMetichecchia
Merge pull request #215 from 18F/revert-filter
e297527 
Update api-data-gov-filter.js
@scottqueen-bixal
updated path on router
ba00a25
@XavierMetichecchia
Merge pull request #216 from 18F/migrate-ga3-to-ga4
7db21c4 
updated path on router
@scottqueen-bixal
remove optional dependency jq
69bcf0e
@scottqueen-bixal
Merge pull request #217 from 18F/poam-package-update
28a7adc 
remove optional dependency jq
@scottqueen-bixal
update package to v2
78634e3
@scottqueen-bixal
add reminder to remove UA when sunset
9b3d8a6
@scottqueen-bixal
first pass to update semantics for v2 routing
05e18bb
@scottqueen-bixal
Merge pull request #220 from 18F/semantic-routing-update-v2
7adfdd0 
Semantic routing update v2
Fix API_SECRET for lower envs
8388e91
@scottqueen-bixal
include message in object response if v1.1
fe2061c
@scottqueen-bixal
update for v2 path
5aab64e
@scottqueen-bixal
Merge pull request #225 from 18F/response-v1-notice
30a4fba 
include message in object response if v1.1
@scottqueen-bixal
pass dbtable on queryDomain instead of version value
530a035
@XavierMetichecchia
Merge pull request #229 from 18F/pass-dbtable-on-querydomain
3a39481 
pass dbtable on queryDomain instead of version value
@scottqueen-bixal
Merge branch 'develop' of github.com:18F/analytics-reporter-api into …
0e8667d 
…update-readme
@scottqueen-bixal
Merge branch 'stage' of github.com:18F/analytics-reporter-api into de…
0ab69de 
…velop
@scottqueen-bixal
include export
05a01dc
@scottqueen-bixal
Merge branch 'develop' of github.com:18F/analytics-reporter-api into …
90da13e 
…develop
@scottqueen-bixal
include deprecated in message
c1eef72
scottqueen-bixal and others added 28 commits January 26, 2024 09:53
@scottqueen-bixal
update to allow table value to be passed
65bd1b0
@scottqueen-bixal
allow for testing multiple routes
a340765
@scottqueen-bixal
toggle back on all db test now that they work in pipeline
9e0831f
@scottqueen-bixal
Merge pull request #226 from 18F/update-readme
e1c5408 
Update Readme
@scottqueen-bixal
include new migration files
8136737
@scottqueen-bixal
remove linter
4dd3798
@scottqueen-bixal
update nmrc and engine to 20
c34f0e2
@scottqueen-bixal
Merge branch 'develop' of github.com:18F/analytics-reporter-api into …
500c681 
…develop
@scottqueen-bixal
remove test node env, and include run migrate
729dcf4
@scottqueen-bixal
update knex script command
c50436d
@XavierMetichecchia
Merge pull request #219 from 18F/develop
1552533 
Version Routing in support of UA to GA4 migration
@levinmr
Add API v1 -> v2 migration docs to README
7846c49
@levinmr
Merge pull request #233 from 18F/add_v2_migration_docs
a59151a 
Add API v1 -> v2 migration docs to README
@levinmr
Add eslint/prettier, run lint:fix on repo
752e0f7
@levinmr
Refactor CI, add lint stage
0eebbaa
@levinmr
Merge pull request #234 from 18F/add_linting
ba90f16 
Add linting support
@levinmr
Add github actions for lint and test
b82d53a
@levinmr
Update local testing setup
e55043e 
Add docker-compose.test.yml for a local test DB
Add mocharc with default files to test so they can be omitted from the test command
Update the knexfile to provide some defaults for dev/test envs
Add lint and test to the README
Disable lint/test in circleCI
@levinmr
Replace CircleCI deployments with Github Actions
da0e27b
@levinmr
Merge pull request #236 from 18F/github_actions
a9a327a 
Add github actions for lint and test
@levinmr
Use env vars correctly in deploy workflow
ef64da4
@levinmr
Remove duplicated caching logic from workflows
f102faf
@levinmr
Add a flag to speed up deploy times
d4846e5
@levinmr
Update README
d7efc9f
@levinmr
Add links to GA4/UA and deprecation warning
a1380ed
@levinmr
Merge pull request #237 from 18F/develop
e4299eb 
Staging deployment
@levinmr
Rename some CI variables for clarity, enhance docs
6997746
@levinmr
Merge pull request #238 from 18F/develop
3b293f0 
Rename some CI variables for clarity, enhance docs
XavierMetichecchia
XavierMetichecchia approved these changes Feb 29, 2024
View reviewed changes
Copy link
Contributor

@XavierMetichecchia XavierMetichecchia left a comment

Choose a reason for hiding this comment

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

Approved

@levinmr levinmr merged commit 63b6495 into master Feb 29, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@XavierMetichecchia XavierMetichecchia XavierMetichecchia approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@levinmr @XavierMetichecchia @scottqueen-bixal