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.

Sign up for GitHub

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

Jump to bottom

URL does not get converted from snake_case responder_url to camelCase responderURL #590

Closed
Jared-Newell-Mobility opened this issue Feb 5, 2024 · 1 comment
Closed

URL does not get converted from snake_case responder_url to camelCase responderURL #590

Jared-Newell-Mobility opened this issue Feb 5, 2024 · 1 comment
Labels
bug Something isn't working For inclusion in release 1.0.0

Comments

@Jared-Newell-Mobility
Copy link
Contributor

In the 2.0.1 spec for AuthorizeRequest and GetCertificateStatusRequest there is a field responderURL that does not get converted from snake_case responder_url to camelCase responderURL. This PR adds handling of this edge case. in SetNetworkProfileRequest there is a field ocppCsmsUrl which is handled correctly but causes problems for a more general solution.

Let me know if the conditionals work for you.

See Original PR #336
Merged
Jared-Newell-Mobility added a commit that referenced this issue Feb 14, 2024
@Jared-Newell-Mobility
URL does not get converted from snake_case responder_url to camelCase…
9fb5769 
… responderURL (#592)

see issue #590
@Jared-Newell-Mobility
Copy link
Contributor Author

Merged so will close

Jared-Newell-Mobility added a commit that referenced this issue Feb 14, 2024
@Jared-Newell-Mobility
Bump to 1.0.0-rc.1 (#611)
dbfb006 
## 1.0.0-rc.1 (2024-02-14)

- [#573](#573) Introduce
Experimental Module For v2.1
- [#547](#547) Feat: Handle
recursively serializing a dataclasses as a dictionary Thanks
[@MacDue](https://github.com/MacDue)
- [#601](#601) Fix case
conversion for soc in non "State of Charge" context
- [#523](#523) The
serialisation of soc to SoC should not occur in camel case if it is
existing at the beginning of a field
- [#515](#515) Update
Readthedocs configuration
- [#602](#602) Correct v2g
serialisation/deserialisation
- [#557](#557) OCPP 2.0.1
Wrong data type in CostUpdated total_cost
- [#564](#564) Add support
For Python 3.11 and 3.12
- [#583](#583) OCPP
v1.6/v2.0.1 deprecate dataclasses from calls and call results with the
suffix 'Payload'
- [#590](#336)
snake_to_camel_case url to URL does not get converted correctly
- [#591](#591)
Camel_to_snake_case doesn't handle v2x correctly
- [#593](#593) Update tests
to use Call and CallResult without the suffix Payload
- [#435](#435) Typo in
CostUpdated Action
- [#577](#577) v2.0.1
AttributeType Enum Corrections
- [#340](#340) 2.0.1
dataclasses have a incorrect types that don't match carnality
- [#519](#519) Typo in
v201.enums.StatusInfoReasonType.invaild_schedule
- [#510](#510) v2.0.1
UnitOfMeasureType - Enums missing and update docstring to allow use for
variableCharacteristics
- [#508](#508) Exception -
OccurrenceConstraintViolationError doc string correction

## DEPRECATED ##
- [#599](#599) v1.6 Action
Enum members corrected IMPORTANT SEE UPGRADE PATH
[#599](#599)
- [#579](#579) v2.0.1 Action
enums corrected - IMPORTANT SEE UPGRADE PATH
[#579](#579)

## BREAKING ##
- [#574](#574) Remove v1.6
deprecated enum members - IMPORTANT see upgrade path
[#574](#574)
- [#498](#498) Remove
support for OCPP 2.0 - IMPORTANT SEE UPGRADE PATH
[#498](#498)
RoaringDev1203 added a commit to RoaringDev1203/Cratus-OCPP that referenced this issue Aug 27, 2024
@RoaringDev1203
URL does not get converted from snake_case responder_url to camelCase…
7bef9f2 
… responderURL (#592)

see issue mobilityhouse/ocpp#590
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working For inclusion in release 1.0.0
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Jared-Newell-Mobility