-
Notifications
You must be signed in to change notification settings - Fork 8
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
Support for Masked Email? #66
Comments
Hi @stefan2904 please, can you help me with my question? |
@stefan2904 Great idea! I wrote some code to support this in #73. Feel free to try that out and let me know what you think. As this is the first provider-specific functionality in jmapc, I've set the |
Applied upstream commits: smkent/cookie-python@224c0e2...a893079 * a893079 Merge pull request #66 from smkent/codecov-action-v3 |\ | * a336750 Update codecov-action to v3 |/ * d4ea3a7 Merge pull request #65 from smkent/python-3.11 * fad136b Add Python 3.11 to builds, and use Python 3.11 for releases
@smkent first of all, thanks a lot for the implementation, and very sorry for my late reply! I finally found some time to experiment with your implementation. Retrieving existing masked emails works fine! def getMaskedEmails(client):
method = MaskedEmailGet(ids=None)
result = client.request(method)
return result.data Unfortunately something is weird when I try to create a masked email. def createMaskedEmail(client):
method = MaskedEmailSet(create=dict(
foo= MaskedEmail(
for_domain="example.com",
description="TEST from python",
email_prefix="stefan"
)
))
result = client.request(method)
return result The interesting thing is: this works, but results in the following error anyway:
(I am also not sure what to set for the Btw, I think the
|
@stefan2904 Thanks for giving this a try and for the detailed report! If you wouldn't mind opening a new issue, it'll help me keep track of it as it'll show up on my dashboard. Thank you! |
Since Fastmail now opened up their JMAP API and included the MaskedEmail management in it, do you think it makes sense to also add that to your library?
The text was updated successfully, but these errors were encountered: