-
Notifications
You must be signed in to change notification settings - Fork 325
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
[WPB-3138] Split brig-schema into lib and exec. #3638
Conversation
ebed3ef
to
c250c48
Compare
eef40d0
to
afd0a1b
Compare
afd0a1b
to
d734d1c
Compare
…-schema-version-brig
A changelog is missing. |
@@ -17,7 +17,7 @@ | |||
-- You should have received a copy of the GNU Affero General Public License along | |||
-- with this program. If not, see <https://www.gnu.org/licenses/>. | |||
|
|||
module V_FUTUREWORK | |||
module Brig.Schema.V_FUTUREWORK |
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.
This module sounds like it needs some love. Can you please create a ticket to address it?
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.
@@ -157,7 +158,7 @@ import Wire.API.User.Identity (Email) | |||
import Wire.API.User.Profile (Locale) | |||
|
|||
schemaVersion :: Int32 | |||
schemaVersion = 79 |
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.
Was this one wrong? Because there's a migration with version 80.
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.
Looks like this was outdated, could've been due to MLS and develop getting merged?
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.
I just hope there won't be a problem in any installation due to this, as in a wrong thing was in place and it worked, but this made it right, but actually broke it.
https://wearezeta.atlassian.net/browse/WPB-3138
Checklist
changelog.d