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

Remove unused remnants of payment page #642

Merged
merged 1 commit into from Aug 25, 2015
Merged

Remove unused remnants of payment page #642

merged 1 commit into from Aug 25, 2015

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Aug 25, 2015

As far as I know this Barclays payment page hasn’t been used for a long time – though I can't determine when it stopped being a thing. I can't find any further references to "barclays" or "epdq", or any of the other terms used in this old payments template. Can anyone provide some context about when other parts of this were removed?

Old commits relating to these files:
b63f325
fd32c16

Some more history:
https://github.com/alphagov/static/commits/fd32c161e5b35ac6dac3e938087843a444cf49bb/public/templates

cc @edds @alext

Barclays payment page hasn’t been used for a long time.
@bradwright
Copy link
Contributor

@bradwright bradwright commented Aug 25, 2015

It stopped being a thing when FCO services took over our hosting of those pages (late 2014).

@alext
Copy link
Contributor

@alext alext commented Aug 25, 2015

This would have been used back when transaction_wrappers was a thing (https://github.com/gds-attic/transaction-wrappers). With the move of these services to a service domain, this would have no longer been used.

edds added a commit that referenced this pull request Aug 25, 2015
Remove unused remnants of payment page
@edds edds merged commit aa275b9 into master Aug 25, 2015
1 check passed
1 check passed
default "Build #580 succeeded on Jenkins"
Details
@edds edds deleted the remove-barclays branch Aug 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.