Skip to content
This repository has been archived by the owner on Sep 29, 2020. It is now read-only.

Contribution-New Contact Form Plugin for Pawtucket #6

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

zi94sm65
Copy link

No description provided.

@collectiveaccess
Copy link
Owner

Can we make the "to email" configurable rather than hard-coding it in the code itself?
And we should probably figure out why the capitalization is an issue. Are you running this on a Mac when you test?

@zi94sm65
Copy link
Author

Sure, making the "to email" configurable is a great idea; however since i'm new to CA i was not sure how to do this.

I'm not sure why there were issues with the capitalization of file names but figured it had something to do with Zend naming conventions (i know Codeigniter can be picky about capitalization and how you name classes) but i don't use Zend so i was not sure if they had any rules.

I'm running a localhost on my Mac and i kept getting "invalid controller path" errors until i de-capitalized front half of file names: formContactPlugin had to become formcontactPlugin and formContactController had to become formcontactController to get it to work. Then when i uploaded it our production server it would not work unless formcontactController became FormcontactController. I figured other people might run into the same type of issues, so that is why i left the note in there about possibly having to change the capitalization of file names.


From: CollectiveAccess [email protected]
To: collectiveaccess/pawtucket [email protected]
Cc: zi94sm65 [email protected]
Sent: Monday, March 18, 2013 3:44 PM
Subject: Re: [pawtucket] Contribution-New Contact Form Plugin for Pawtucket (#6)

Can we make the "to email" configurable rather than hard-coding it in the code itself?
And we should probably figure out why the capitalization is an issue. Are you running this on a Mac when you test?

Reply to this email directly or view it on GitHub.

@collectiveaccess
Copy link
Owner

You can just put the email in the config file and pull it via the Configuration class.

Macs are case insensitive (unless you specifically use a specific non-default file system), Linux is case-sensitive. I'll look at your code and see if I can find the problem.

Thanks for contributing!

@zi94sm65
Copy link
Author

Made requested changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants