Skip to content

Conversation

@jo-wojtowicz
Copy link

We were recommended by the Airship support to change the way we associate email channels with named user IDs. This operation is not currently supported by the Airship's Ruby Library, so we decided to add it to our fork.

Connects to the issue: https://github.com/OLIOEX/community-team/issues/1188

https://docs.airship.com/api/ua/index.html#operation-api-named_users-associate-post

Co-authored-by: name <evie@olioex.com>
Co-authored-by: name <arron.kukadia@olioex.com>
@lloydwatkin
Copy link
Member

This should be contributed to Airship :)

@lloydwatkin
Copy link
Member

You are missing documentation updates here 👍

@jo-wojtowicz
Copy link
Author

jo-wojtowicz commented Jun 28, 2022

@lloydwatkin regarding contributing to Airship, there is this: urbanairship#132 PR containing current changes - would that be sufficient? :)

Regarding your comment:

You are missing documentation updates here 👍

I am not sure if I understand correctly, do you mean we should add some documentation for these changes? Or that there were some documentation changes for the Airship library that we have not taken into account while implementing our changes?

Thank you very much for your help :)

@lloydwatkin
Copy link
Member

@majhogee take a look in the docs directory. You'll need to add documentation for this change

@lloydwatkin
Copy link
Member

@lloydwatkin regarding contributing to Airship, there is this: urbanairship#132 PR containing current changes - would that be sufficient? :)

Yes but please remove internal references to our repositories ASAP

@jo-wojtowicz
Copy link
Author

Thank you @lloydwatkin, I've removed the reference to our repo, I will chat with @eviepolds about updating the docs :)

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.

3 participants