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

In contact:addrType the elements sp and pc (contact::Address fields province and postal_code) are optional #13

Closed
kmkaplan opened this issue Mar 31, 2023 · 3 comments

Comments

@kmkaplan
Copy link
Contributor

The elements sp, pc are optional but the struct Address in module contact makes them compulsory.

Here is the schema definition from RFC 5733, page 31

<complexType name="addrType">
  <sequence>
    <element name="street" type="contact:optPostalLineType"
     minOccurs="0" maxOccurs="3"/>
    <element name="city" type="contact:postalLineType"/>
    <element name="sp" type="contact:optPostalLineType"
     minOccurs="0"/>
    <element name="pc" type="contact:pcType"
     minOccurs="0"/>
    <element name="cc" type="contact:ccType"/>
  </sequence>
</complexType>

Should I fix this and submit PR?

This was originaly filed as masalachai/epp-client#96

@djc
Copy link
Owner

djc commented Mar 31, 2023

Yes, please!

@kmkaplan
Copy link
Contributor Author

Ok, I'll start on monday.
Note that this will imply some API changes.

@djc
Copy link
Owner

djc commented Mar 31, 2023

Ok, I'll start on monday.

Note that this will imply some API changes.

That's not a problem. For our internal usage we're doing Git dependencies for now, we can put releases out if you have a need for that.

kmkaplan added a commit to kmkaplan/instant-epp that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new` and the type of
fields `contact::InfoData::voice`.

Fixes issue djc#13.
kmkaplan added a commit to kmkaplan/instant-epp that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new` and the type of
fields `contact::InfoData::voice`.

Fixes issue djc#13.
kmkaplan added a commit to kmkaplan/instant-epp that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new`,
`contact::ContactCreate::new` and the type of fields
`contact::InfoData::voice` and `contact::ContactCreateRequest::voice`.

Fixes issue djc#13.i
kmkaplan added a commit to kmkaplan/instant-epp that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new`,
`contact::ContactCreate::new` and the type of fields
`contact::InfoData::voice` and `contact::ContactCreateRequest::voice`.

Fixes issue djc#13.
kmkaplan added a commit to kmkaplan/instant-epp that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new`,
`contact::ContactCreate::new` and the type of fields
`contact::InfoData::voice` and `contact::ContactCreateRequest::voice`.

Fixes issue djc#13.
djc pushed a commit that referenced this issue Apr 3, 2023
This changes the signature for `contact::Address::new`,
`contact::ContactCreate::new` and the type of fields
`contact::InfoData::voice` and `contact::ContactCreateRequest::voice`.

Fixes issue #13.
@kmkaplan kmkaplan closed this as completed Apr 3, 2023
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

No branches or pull requests

2 participants