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.

Sign up for GitHub

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

Jump to bottom

MM-60705 DB migration: added update_at column to draft table for mobile #8237

Merged
merged 4 commits into from
Oct 15, 2024
Merged

MM-60705 DB migration: added update_at column to draft table for mobile #8237

merged 4 commits into from
Oct 15, 2024

Conversation

Rajat-Dabade
Copy link
Contributor

Summary

Added update_at column to the draft table for mobile.

Ticket Link

https://mattermost.atlassian.net/browse/MM-60705

Checklist

  • Added or updated unit tests (required for all new features)
  • Has UI changes
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.
  • Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on:
Andriod: 15 and iOS: 17.5

Release Note

Added a new column `update_at` to the Drafts Table.

@Rajat-Dabade Rajat-Dabade added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Sep 30, 2024
@Rajat-Dabade Rajat-Dabade self-assigned this Sep 30, 2024
@larkox
Copy link
Contributor

larkox commented Sep 30, 2024

@Rajat-Dabade I think you are still missing the docs updates for this PR

@Rajat-Dabade
Copy link
Contributor Author

@larkox will be updating it today.

@yasserfaraazkhan yasserfaraazkhan added the Build App for iOS Build the mobile app for iOS label Oct 1, 2024
enahum
enahum requested changes Oct 10, 2024
View reviewed changes
app/components/post_draft/index.ts Outdated Show resolved Hide resolved
app/database/operator/server_data_operator/transformers/post.ts Outdated Show resolved Hide resolved
@larkox larkox added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Oct 15, 2024
@Rajat-Dabade Rajat-Dabade merged commit 647b8c8 into main Oct 15, 2024
12 checks passed
@Rajat-Dabade Rajat-Dabade deleted the migration-drafts branch October 15, 2024 14:19
@amyblais amyblais added this to the v2.23.0 milestone Oct 15, 2024
@amyblais amyblais added the Docs/Needed Requires documentation label Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yasserfaraazkhan yasserfaraazkhan yasserfaraazkhan approved these changes

@enahum enahum enahum approved these changes

@larkox larkox larkox approved these changes

Assignees
No one assigned
Labels
4: Reviews Complete All reviewers have approved the pull request Build App for iOS Build the mobile app for iOS Docs/Needed Requires documentation release-note
Projects
None yet
Milestone
v2.23.0
Development

Successfully merging this pull request may close these issues.
6 participants
@Rajat-Dabade @larkox @enahum @yasserfaraazkhan @amyblais @mm-cloud-bot