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

Contribution-New Contact Form Plugin for Pawtucket#6

Open
zi94sm65 wants to merge 13 commits intocollectiveaccess:masterfrom
zi94sm65:master
Open

Contribution-New Contact Form Plugin for Pawtucket#6
zi94sm65 wants to merge 13 commits intocollectiveaccess:masterfrom
zi94sm65:master

Conversation

@zi94sm65
Copy link
Copy Markdown

No description provided.

@collectiveaccess
Copy link
Copy Markdown
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
Copy Markdown
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 notifications@github.com
To: collectiveaccess/pawtucket pawtucket@noreply.github.com
Cc: zi94sm65 zi94sm65@yahoo.com
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
Copy Markdown
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
Copy Markdown
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