Skip to content

Comments

♻️ Process updated list from Spring#7

Merged
zwaans merged 5 commits intomasterfrom
feature/process-updated-list-from-spring
Jun 13, 2025
Merged

♻️ Process updated list from Spring#7
zwaans merged 5 commits intomasterfrom
feature/process-updated-list-from-spring

Conversation

@M4tini
Copy link
Member

@M4tini M4tini commented Jun 11, 2025

https://myparcelcom.slack.com/archives/CT25ZMR1P/p1749113775502109

Processed the updated list from Spring (provided Excel is the same as provided PDF).

We only support 1 final-mile carrier per country, so any other records are ignored.
(we can only support this when we have a reliable way to recognize tracking code patterns)

Other quality of life improvements in this PR:

  • Updated PHP from 8.0 to 8.2
  • Updated composer from 9 to 11
  • Use docker compose for local development

@M4tini M4tini force-pushed the feature/process-updated-list-from-spring branch from 51bd49e to ec4b906 Compare June 11, 2025 18:55
@zwaans zwaans self-assigned this Jun 13, 2025
@daniel-myparcel daniel-myparcel requested a review from zwaans June 13, 2025 08:28
@@ -0,0 +1,9 @@
FROM php:8.2-cli
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets store this in the same folder as we do for the other projects. How does this get published to packagist again?

Copy link
Member Author

Choose a reason for hiding this comment

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

Moved! 🚚 And I think it automatically publishes when we make a tag in this repo

@M4tini M4tini requested a review from zwaans June 13, 2025 09:30
@zwaans zwaans merged commit b245678 into master Jun 13, 2025
1 check passed
@zwaans zwaans deleted the feature/process-updated-list-from-spring branch June 13, 2025 09:59
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.

2 participants