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.

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

Notify administrator when employer updates job listing #1438

Merged
merged 2 commits into from
Apr 12, 2018

Conversation

jom
Copy link
Member

@jom jom commented Apr 12, 2018

Fixes #1437
Based from #1434 (must be merged first -- rebased)

Changes proposed in this Pull Request:

  • Sends a new notification when a job is updated by an employer.

Testing instructions:

  • Update a job from the [job_dashboard] shortcode and verify the admin is sent a notice.

Proposed changelog entry for your changes:

  • Notify site administrators when a job has been updated by an employer.

@jom jom added this to the 1.31.0 milestone Apr 12, 2018
@jom jom requested review from alexsanford and donnapep April 12, 2018 10:15
@jom jom force-pushed the add/update-job-email branch from 6692120 to 4c198a3 Compare April 12, 2018 12:33
@jom jom force-pushed the add/update-job-email branch from 4c198a3 to 263a1af Compare April 12, 2018 14:07
Copy link
Contributor

@alexsanford alexsanford left a comment

Choose a reason for hiding this comment

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

🚀

@jom jom merged commit 341fb77 into master Apr 12, 2018
@jom jom deleted the add/update-job-email branch April 12, 2018 15:07
@jom jom mentioned this pull request Apr 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants