Skip to content

Rename the chargable to pay to use - #4847

Open
lindseydew wants to merge 145 commits into
mainfrom
ld/pay-to-use
Open

Rename the chargable to pay to use#4847
lindseydew wants to merge 145 commits into
mainfrom
ld/pay-to-use

Conversation

@lindseydew

@lindseydew lindseydew commented Jul 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What does this change?

This changes the name of Chargable class from Chargeable supplied / on spec to Pay to use

How should a reviewer test this change?

How can success be measured?

Who should look at this?

image image

Tested? Documented?

  • locally by committer
  • locally by Guardian reviewer
  • on the Guardian's TEST environment
  • relevant documentation added or amended (if needed)

lindseydew added 30 commits June 25, 2026 10:28
@lindseydew
Deprecate kinesis v1
85f1cbb
@lindseydew
Merge branch 'ld/sqs-sns-upgrade' into ld/kinesis-upgrade
0477ec0
@lindseydew
Update the querybyusageid
e438c7e
@lindseydew
Add the query by image id
63ab071
@lindseydew
Add an update builder
1360bdd
@lindseydew
Add the update method
8fb195e
@lindseydew
Remove unused dynamo method
05f2dab
@lindseydew
Rename from v2
3c56bd0
@lindseydew
Add tests for the expression builder
4a9f49b
@lindseydew
Add tests for the toExpression
4857a49
@lindseydew
Use a dynamo typing system for the case classes
2f01380
@lindseydew
Add a toDynamoMap in
78f013d
@lindseydew
Add a comprehensive test for all fields
fa89e3b
@lindseydew
Use the toDynamoMap
0136b09
@lindseydew
Update the remove operation
ae9babb
@lindseydew
Use the correct index name
b2b5ba2
@lindseydew
Use the correct name
1e07819
@lindseydew
Update the image name
75aa9a6
@lindseydew
Fix the metadata read endpoint
9d7538c
@lindseydew
Merge branch 'main' into ld/aws-dynamo-upgrade-4
ad94920
@lindseydew
Merge branch 'ld/aws-dynamo-upgrade-2' into ld/aws-dynamo-upgrade-4
c89b2ad
@lindseydew
Merge branch 'ld/aws-dynamo-upgrade-4' of github.com:guardian/grid in…
d5d65e5 
…to ld/aws-dynamo-upgrade-4
@lindseydew
Add a test for v1 transformation
7a47597
@lindseydew
Add a test for the v2 version with simple fields
1cc3c8e
@lindseydew
Add test for complex fields with map string
03a71f7
@lindseydew
Update the attribute value map
2287adb
@lindseydew
Add tests for the extra fields
5cd0a81
@lindseydew
Merge branch 'main' into ld/aws-dynamo-upgrade-4
40d9df4
@lindseydew
Merge branch 'main' into ld/aws-dynamo-upgrade-4
e7c3f46
@lindseydew
Add the kinesis client to common lib
68c4240
lindseydew and others added 30 commits August 5, 2026 10:27
@lindseydew
Merge pull request #4836 from guardian/ld/upgrade-aws-s3-part6
6a53515 
Upgrade S3 - Part 6
@lindseydew
Merge pull request #4837 from guardian/ld/upgrade-aws-s3-part7
40b4d96 
Upgrade S3 - Part 7
@dependabot
chore(deps): bump actions/setup-node from 6 to 7
f732aa0 
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 6 to 7.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@v6...v7)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@hk-15
Pin actions/setup-node to commit hash
c822b8a
@hk-15
Merge pull request #4809 from guardian/dependabot/github_actions/acti…
f4c4418 
…ons/setup-node-7

chore(deps): bump actions/setup-node from 6 to 7
@lindseydew
First pass at the script
92785e4
@lindseydew
Track the progress of the script
2e61484
@lindseydew
Read from the csv
edbca8a
@lindseydew
Set a stage variable
1e970c2
@lindseydew
Infer the domain from the stage set
764d881
@lindseydew
Consolidate log statements
582f3e6
@lindseydew
Merge pull request #4856 from guardian/ld/script-alamy
bd0ff4a 
Script Alamy changes
@lindseydew
Build the presigner client for a local run
4623216
@lindseydew
Merge pull request #4838 from guardian/ld/upgrade-aws-s3-part8
f000e46 
Upgrade S3 - Part 8
@lindseydew
Add logic for setting a label
4232830
@lindseydew
Merge branch 'main' into ld/fix-local-run
b610d38
@lindseydew
Merge pull request #4857 from guardian/ld/fix-local-run
15fb008 
Build the presigner client for a local run
@hk-15
Add script to bulk remove images from a given collection
4b64e87
@hk-15
Merge pull request #4864 from guardian/hk/bulk-remove-from-collection
93ba4a3 
Add script to bulk remove images from a given collection
@lindseydew @junyuanxue
Update scripts/src/main/scala/com/gu/mediaservice/scripts/AlamyCleanU…
ac508cb 
…p.scala

Co-authored-by: Junyuan Xue <junyuanxue.is@gmail.com>
@lindseydew @hk-15
Update scripts/src/main/scala/com/gu/mediaservice/scripts/AlamyCleanU…
2c20734 
…p.scala

Co-authored-by: Holly Knox <168672047+hk-15@users.noreply.github.com>
@lindseydew
Merge pull request #4839 from guardian/ld/upgrade-aws-s3-part9
b5c63d5 
S3 Upgrade - Part 9
@lindseydew
Merge pull request #4863 from guardian/ld/alamy-set-label
340be42 
Add logic for setting a label
@lindseydew
Rename the chargable to pay to use
bab38e7
@lindseydew
Put the Chargeable name change behind config switch
8c73464
@lindseydew
Update existing test config
5406e6e
@lindseydew
Add tests for Chargeable name
070ce03
@lindseydew
Port the media api tests to use common fixtures
fefca85
@lindseydew
Merge branch 'ld/pay-to-use' of github.com:guardian/grid into ld/pay-…
0d33aa5 
…to-use
@lindseydew
Split out the elastic search config in ImageResponse
5f100ef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@junyuanxue junyuanxue junyuanxue approved these changes

Assignees

No one assigned

Labels

donotmerge feature Departmental tracking: work on a new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@lindseydew @junyuanxue @ellenmuller @dblatcher @hk-15