-
Notifications
You must be signed in to change notification settings - Fork 326
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
Tidy up partner role. #592
Conversation
all integration tests pass locally. |
@@ -6,6 +6,8 @@ | |||
{-# LANGUAGE TypeFamilies #-} | |||
{-# LANGUAGE ViewPatterns #-} | |||
|
|||
{-# OPTIONS_GHC -Wno-orphans #-} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you write what instance this is needed for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops, it's not. the instance popped up there briefly as quantum noise only. ok if i'll remove it in a separate PR to get this merged two hours earlier?
where | ||
invmetaJson :: (UserId, UTCTimeMillis) -> Value | ||
invmetaJson (by, at) = object [ "by" .= by, "at" .= at ] | ||
[ "created_by" .= (fst <$> _invitation m) ] <> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a non-breaking change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All changes have been agreed on with the other teams.
Oh okay.
@@ -515,13 +513,16 @@ parseTeamMember :: Value -> Parser TeamMember | |||
parseTeamMember = withObject "team-member" $ \o -> | |||
TeamMember <$> o .: "user" | |||
<*> o .: "permissions" | |||
<*> (parseInv =<< (o .:? "invited")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the "invited" field is not present anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All changes have been agreed on with the other teams.
Oh also okay.
Sure
…On Wed, Jan 23, 2019, 17:01 fisx ***@***.*** wrote:
***@***.**** commented on this pull request.
------------------------------
In libs/brig-types/test/unit/Test/Brig/Types/User.hs
<#592 (comment)>:
> @@ -6,6 +6,8 @@
{-# LANGUAGE TypeFamilies #-}
{-# LANGUAGE ViewPatterns #-}
+{-# OPTIONS_GHC -Wno-orphans #-}
oops, it's not. the instance popped up there briefly as quantum noise
only. ok if i'll remove it in a separate PR to get this merged two hours
earlier?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#592 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABc-aryx39yBgBs70XX3DPglgiyH7RIMks5vGIdYgaJpZM4aPET1>
.
|
* Cleanup as promised here: #592 (comment) * Swagger: update docs, remove dead code.
This changes will mess with staging only, since none of the features touched have seen master yet. All changes have been agreed on with the other teams.