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

Added post information to feedback email #1

Merged
merged 1 commit into from
Sep 9, 2016

Conversation

mrlynn
Copy link
Contributor

@mrlynn mrlynn commented Aug 16, 2016

I added address, city, state, zip to the feedback email. This is because the url that is placed at the bottom of the email refers to a post id which changes for us nightly as we re-import meetings from our source database.

@FlipperPA
Copy link
Contributor

Sounds like a good change, of course I'm biased! :) Congrats to @mrlynn on his first open source project pull request!

@joshreisner joshreisner merged commit 4cb5244 into code4recovery:master Sep 9, 2016
@joshreisner
Copy link
Contributor

Hey yall, sorry I didn't see this! I need to check on my notifications settings.

@joshreisner
Copy link
Contributor

I can't figure out how to turn pull request notification emails on, so lmk if you make any more, and please make as many as you want.

I've committed 2.5.2 to Wordpress with this change in it.

@FlipperPA
Copy link
Contributor

Hey @joshreisner! I believe you just need to WATCH the project; I'm guessing you're not getting notified since it is within the "intergroup" group rather than your personal space. Mike and I are coming down to NAATW in November; I'm very much looking forward to meeting you!

@joshreisner
Copy link
Contributor

Oh that's great to hear, looking forward to meeting you too @FlipperPA.

joshreisner pushed a commit that referenced this pull request Oct 11, 2017
joshreisner added a commit that referenced this pull request Oct 11, 2017
Merge pull request #1 from meeting-guide/master
tech2serve pushed a commit that referenced this pull request Oct 26, 2020
@ncd12aa ncd12aa mentioned this pull request Jun 1, 2021
@masterlifter masterlifter mentioned this pull request Dec 18, 2021
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.

3 participants