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

Update xc40_bev.json #303

Merged
merged 7 commits into from
Jan 6, 2025
Merged

Update xc40_bev.json #303

merged 7 commits into from
Jan 6, 2025

Conversation

jomud9
Copy link
Contributor

@jomud9 jomud9 commented Jan 3, 2025

  1. Create new xc40_bev.json for XC40/C40 Pure Electric (BEV). It should work for model MY21 to MY25.
  2. Add new PID for Odometer

Change XC40 to XC40 BEV, refer as "Pure Electric Vehicle"
For XC40 BEV - Pure Electric.

This file should work for XC40/C40 Pure Electric from MY21 to MY25
Add Odometer
@meatpiHQ
Copy link
Owner

meatpiHQ commented Jan 5, 2025

@jomud9 Thanks for that, great work. Can you please put jekyll-gh-pages.yml in a separate PR.

Btw what does jekyll-gh-pages.yml do?

@jomud9
Copy link
Contributor Author

jomud9 commented Jan 6, 2025

@jomud9 Thanks for that, great work. Can you please put jekyll-gh-pages.yml in a separate PR.

Btw what does jekyll-gh-pages.yml do?

I remembered it is file generated when I tried to run deployment workflow, which I guess not intended.

anyway I hope you could help to

  1. Merge my update on json files
  2. Ignore that yml

If it is too much work, let me know and I will just re create that PR for json files update :-)

Remove unrelated yml file created accidentally
@meatpiHQ meatpiHQ merged commit 45440b1 into meatpiHQ:main Jan 6, 2025
1 check failed
@meatpiHQ
Copy link
Owner

meatpiHQ commented Jan 6, 2025

@jomud9 Looks like the profile had some json errors. Please fix and make another PR.

@jomud9
Copy link
Contributor Author

jomud9 commented Jan 6, 2025

@jomud9 Looks like the profile had some json errors. Please fix and make another PR.

Please help and check PR#308. Thanks

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