Skip to content

Editor review complete email now contains additional information and a direct link#4250

Closed
ajrbyers wants to merge 2 commits into
masterfrom
1829-enhancement
Closed

Editor review complete email now contains additional information and a direct link#4250
ajrbyers wants to merge 2 commits into
masterfrom
1829-enhancement

Conversation

@ajrbyers

@ajrbyers ajrbyers commented Jun 4, 2024

Copy link
Copy Markdown
Member

There were 6 close variants of this text.

Closes #1829

@mauromsl mauromsl requested a review from joemull June 26, 2024 13:46
@mauromsl mauromsl assigned joemull and unassigned mauromsl Jun 26, 2024

@joemull joemull left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick on copywriting.

Comment thread src/core/migrations/0095_auto_20240604_1408.py Outdated
@joemull joemull assigned ajrbyers and unassigned joemull Jun 27, 2024
@ajrbyers ajrbyers assigned joemull and unassigned ajrbyers Jul 11, 2024
@ajrbyers ajrbyers changed the title Editor review complete email now contains additionally information and a direct link Editor review complete email now contains additional information and a direct link Jul 12, 2024
@joemull

joemull commented Jul 15, 2024

Copy link
Copy Markdown
Member

@ajrbyers this will now cause a migration mismatch. Could we rebase it locally and change the migration dependency to 0095_auto_20240605_1024?

@joemull joemull assigned ajrbyers and unassigned joemull Jul 15, 2024
@ajrbyers

Copy link
Copy Markdown
Member Author

Closed this in favour of #4324

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.

Review Response to Editor Email

3 participants