Skip to content

Commit bcd3638

Browse files
committed
Revert "Merge branch 'main' into scc-5050-2"
This reverts commit bd098d5, reversing changes made to 5990165.
1 parent bd098d5 commit bcd3638

22 files changed

+79
-431
lines changed

.github/dependabot.yml

Lines changed: 0 additions & 10 deletions
This file was deleted.

.github/workflows/test-and-deploy.yml

Lines changed: 3 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -15,32 +15,12 @@ jobs:
1515
run: npm ci
1616
- name: Unit Tests
1717
run: npm test
18-
integration-test-qa:
19-
permissions:
20-
id-token: write
21-
contents: read
22-
runs-on: ubuntu-latest
23-
needs: tests
24-
if: github.ref == 'refs/heads/qa'
25-
steps:
26-
- uses: actions/checkout@v4
27-
- name: Set Node version
28-
uses: actions/setup-node@v4
29-
with:
30-
node-version-file: '.nvmrc'
31-
- name: Install dependencies
32-
run: npm ci
33-
- name: Start service
34-
run: ENV=qa npm start &
35-
- name: Run tests
36-
run: npm run test-integration
3718
deploy-qa:
3819
permissions:
3920
id-token: write
4021
contents: read
4122
runs-on: ubuntu-latest
42-
needs:
43-
- tests
23+
needs: tests
4424
if: github.ref == 'refs/heads/qa'
4525
steps:
4626
- name: Checkout repo
@@ -51,6 +31,7 @@ jobs:
5131
with:
5232
role-to-assume: arn:aws:iam::946183545209:role/GithubActionsDeployerRole
5333
aws-region: us-east-1
34+
5435
- name: Log in to ECR
5536
id: login-ecr
5637
uses: aws-actions/amazon-ecr-login@v1
@@ -79,8 +60,7 @@ jobs:
7960
id-token: write
8061
contents: read
8162
runs-on: ubuntu-latest
82-
needs:
83-
- tests
63+
needs: tests
8464
if: github.ref == 'refs/heads/qa2'
8565
steps:
8666
- name: Checkout repo

app.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ app.init = async () => {
3333

3434
require('./lib/resources')(app)
3535
require('./lib/subjects')(app)
36-
require('./lib/contributors')(app)
3736
require('./lib/vocabularies')(app)
3837

3938
// routes

config/qa.env

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
ENCRYPTED_ELASTICSEARCH_URI=AQECAHh7ea2tyZ6phZgT4B9BDKwguhlFtRC6hgt+7HbmeFsrsgAAAJYwgZMGCSqGSIb3DQEHBqCBhTCBggIBADB9BgkqhkiG9w0BBwEwHgYJYIZIAWUDBAEuMBEEDMIkDoQ9C/cCDCAq1wIBEIBQ+L3OgUGeOW9rs1CWkhpBjwM4LbbVRFIWedqew4UXIeSNMJ8cO9SNe4YGCUIoKwCDYt7W7ip3VtDRRRMVvz6QJw+Eg8ugTMVs2pbNFGNvaAQ=
2-
3-
ENCRYPTED_RESOURCES_INDEX=AQECAHh7ea2tyZ6phZgT4B9BDKwguhlFtRC6hgt+7HbmeFsrsgAAAHIwcAYJKoZIhvcNAQcGoGMwYQIBADBcBgkqhkiG9w0BBwEwHgYJYIZIAWUDBAEuMBEEDPMBVNbSFDq16QAs4AIBEIAvHrJZjGewR7g4oT5oifQUDGTj2SgYibnrhU05uBatHEVYz/mOawAVrjt/1oxPqv4=
4-
BROWSE_INDEX=browse-qa-2026-01-15
2+
RESOURCES_INDEX=resources-2025-07-07
3+
SUBJECTS_INDEX=browse-qa-2025-10-27
54
ENCRYPTED_ELASTICSEARCH_API_KEY=AQECAHh7ea2tyZ6phZgT4B9BDKwguhlFtRC6hgt+7HbmeFsrsgAAAJ4wgZsGCSqGSIb3DQEHBqCBjTCBigIBADCBhAYJKoZIhvcNAQcBMB4GCWCGSAFlAwQBLjARBAx+kryf2KUmGdBYD9sCARCAV3ygz3eXIdq8JX/wpG9JRWlTNMRcpNE1qT0zNlN4t+ZvXEoedLQa/3p1YjgHw06GIAdA9xtkMV4eH9a1K8uCvjP8XxxNKekcMj59TlResnu9QF3r7pGXuQ==
65

76
ENCRYPTED_SCSB_URL=AQECAHh7ea2tyZ6phZgT4B9BDKwguhlFtRC6hgt+7HbmeFsrsgAAAH8wfQYJKoZIhvcNAQcGoHAwbgIBADBpBgkqhkiG9w0BBwEwHgYJYIZIAWUDBAEuMBEEDBKllElmWYLxGOGopQIBEIA8JJyKde/8m8iCJGKR5D8HoTJhXHeyvw9eIDeuUNKiXLfJwoVz+PDAZSxkCQtM9O91zGhXbe3l6Bk1RlYJ

config/test.env

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
ELASTICSEARCH_URI=encrypted-elasticsearch-uri
22
RESOURCES_INDEX=test-resources-index
3-
BROWSE_INDEX=test-browse-index
3+
SUBJECTS_INDEX=test-subjects-index
44

55
SCSB_URL=encrypted-scsb-url
66
SCSB_API_KEY=encrypted-scsb-api-key

lib/api-request.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,6 @@ class ApiRequest {
7373
return this.params.subject_prefix
7474
}
7575

76-
hasContributorRole () {
77-
return this.params.role && this.params.filters.contributorLiteral
78-
}
79-
8076
static fromParams (params) {
8177
return new ApiRequest(params)
8278
}

lib/contributors.js

Lines changed: 0 additions & 121 deletions
This file was deleted.

lib/elasticsearch/browse-utils.js

Lines changed: 0 additions & 29 deletions
This file was deleted.

lib/elasticsearch/config.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,8 @@ const AGGREGATIONS_SPEC = {
124124
buildingLocation: { terms: { field: 'buildingLocationIds' } },
125125
subjectLiteral: { terms: { field: 'subjectLiteral.raw' } },
126126
language: { terms: { field: 'language_packed' } },
127+
contributorLiteral: { terms: { field: 'contributorLiteral.raw' } },
128+
creatorLiteral: { terms: { field: 'creatorLiteral.raw' } },
127129
collection: { terms: { field: 'collectionIds' } }
128130
}
129131

lib/elasticsearch/elastic-query-builder.js

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -92,10 +92,6 @@ class ElasticQueryBuilder {
9292
if (this.request.hasSubjectPrefix()) {
9393
this.applySubjectPrefix()
9494
}
95-
96-
if (this.request.hasContributorRole()) {
97-
this.applyContributorRole()
98-
}
9995
}
10096

10197
/**
@@ -179,13 +175,6 @@ class ElasticQueryBuilder {
179175
])
180176
}
181177

182-
/**
183-
* Concat contributor + role if role param is provided
184-
*/
185-
applyContributorRole () {
186-
this.query.addMust(termMatch('contributorRoleLiteral', this.request.params.filters.contributorLiteral + '||' + this.request.params.role))
187-
}
188-
189178
/**
190179
* Build ES query for standard_number searches
191180
*/

0 commit comments

Comments
 (0)