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

feat(shipper): update intelcom, ups and amazon with canadian french strings #954

Merged
merged 8 commits into from
Jul 18, 2024

Conversation

Bluscream
Copy link

Basically redone #842 for the correct branch

  • Add french strings to amazon + canadian french AMAZON_LANG
  • Add french string to ups
  • IntelCom section with french contents

Breaking change

Proposed change

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests
  • Documentation update
  • Adds a new shipper
  • Update existing shipper

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:

Basically redone moralmunky#842 for the correct branch

- Add french strings to amazon + canadian french AMAZON_LANG
- Add french string to ups
- IntelCom section with french contents
@Bluscream Bluscream changed the title Extended consts (closes #842) fix: Extended consts (closes #842) Jul 17, 2024
@firstof9 firstof9 changed the title fix: Extended consts (closes #842) feat(shipper): update intelcom, ups and amazon with canadian french strings Jul 17, 2024
@firstof9 firstof9 added the enhancement Changes that reduce code complexity or speed things up label Jul 17, 2024
To hopefully fix the issue tox's linter has with this PR
"update-bestelling@",
"versandbestaetigung@"
]
AMAZON_EMAIL = ["order-update@", "update-bestelling@", "versandbestaetigung@"]
Copy link
Author

@Bluscream Bluscream Jul 18, 2024

Choose a reason for hiding this comment

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

what a dumb change, but apparently the linter is happy now, so be it ig

Bluscream and others added 2 commits July 18, 2024 04:30
linter helped me find that lol
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.43%. Comparing base (8608d22) to head (a38d2a5).
Report is 28 commits behind head on dev.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #954      +/-   ##
==========================================
- Coverage   95.07%   93.43%   -1.64%     
==========================================
  Files           8        8              
  Lines        1482     1554      +72     
==========================================
+ Hits         1409     1452      +43     
- Misses         73      102      +29     
Files Coverage Δ
custom_components/mail_and_packages/const.py 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

@firstof9 firstof9 merged commit 26f0ac1 into moralmunky:dev Jul 18, 2024
5 checks passed
@firstof9
Copy link
Collaborator

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Changes that reduce code complexity or speed things up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants