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

addressing issues #20 and #17 #30

Closed
wants to merge 1 commit into from

Conversation

anewton1998
Copy link
Collaborator

this addresses #20 and #17

Copy link
Contributor

@mwullink mwullink left a comment

Choose a reason for hiding this comment

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

made a suggestion for text

@@ -29,7 +29,10 @@ of this working group, but direct compatibility of RPP with EPP is not
a goal.
The REGEXT working group is chartered to maintain and standardize extensions to EPP, therefore
extensions or changes for EPP are explicitly out of scope for the RPP working group.
The RPP working group may however consider possibilities of mapping of data objects, operations and extensions from EPP to RPP for both clients and servers.
The RPP working group may however consider possibilities of mapping data objects, operations, command and response,
and other EPP extension mechanisms to RPP elements and features for both clients and servers. The working group
Copy link
Contributor

Choose a reason for hiding this comment

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

By using the text "and other EPP extension mechanisms" it seems to me (non native English reader) that the items mentioned before ( data objects, operations, command and response) are are also extension mechanisms

maybe remove: the word "other" to:
The RPP working group may however consider possibilities of mapping EPP data objects, operations, command and response, and extension mechanisms to RPP elements and features for both clients and servers

@anewton1998
Copy link
Collaborator Author

I am closing this in-favor of PR #34

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