-
Notifications
You must be signed in to change notification settings - Fork 45
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
Recaptcha Module and tweaked forms/form_model #14
Open
josev814
wants to merge
31
commits into
brockf:master
Choose a base branch
from
josev814:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added $time variable before $date Added the time variable to the $date's date call Changes $lines[] Date to use $time variable instead of using another function to convert the $date variable Also added T to designate Timezone, since not all users have control of their hosting solutions. Remove else {$lines[] = "Member: None"} since if member doesn't exist in the email it is assumed the person isn't a member already. For systems that aren't using a Member option, the display of it can be confusing to the end client.
updated settings to turn on native spell check added line config.disableNativeSpellChecker = false; to enable spell check
Updated to produces better SEO URL Titles Replaced underscore of url = url.replace(/\s+/g,'_'); with hyphen url = url.replace(/\s+/g,'-'); for better SEO URL Title
…alized with one install
… the bitly_model in order to do so
…t, and added options for bit.ly login and user_key
…irect to the tweet_logs page
…he menu under Reports -> Tweets Sent
…o define the ul child class as something other than children as requested by superlinkx to resolve Issue 10. Use is {menu name="menu_name" show_sub_menus="yes" class="footer_menu" child_class="footer_child another_class yet_another_class"} This results in: <ul class="parent"> <li> parent <ul class="children footer_child another_class yet_another_class"> <li>child</li> </ul> </li> </ul>
…for the publish module
…ly in the admin forms
Updated the cron model to accomodate for the cron hook. Added 2 function to publish and unpublish content depending on the content_date and content_publish_date
… the admin interface.
If using Postmark, only send when the sender’s email is the main site…
…kf#4 on electricfunciton/hero. This is nothing more than a copy of the edit function with the only difference being that it will send to admincp/post/new rather than to edit. Removed the future only option for the date publish, as there shouldn't be an issue with back dating a post.
Had to modify default_routes to include 404_override = 'error/view' Created the Error controller with the view method The view method sets the page title and message to 404 Page Not Found, The page you are looking for could not be found.
Don’t throw warning when logging an array in data_var().
…ow centralized in one function to make it eaiser to update. Changed from using CI string helper to using php's built in uniqid function to generate a random salt.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Can we have the Recaptcha Module included in the default installation?
I also remove the Member:None from the data being inserted on forms, as it seems redundant. If it's not in the record, it would make sense that the user isn't a member.
I added a time variable to be used in the $date and $lines[] Date variables. That way we're getting the date at one time, and not having to pass $date through another function which seems redundant.
I also added T to the date format in order to include Timezone for users that aren't able to set the timezone on their servers.
All of the Electric Function Modules have been included to be centralized.
The Twitter Module was updated to use bit.ly to shorten urls as long as the bitly settings are filled out in the admin configuration area. A table has been created to log all tweets sent, along with a report added.
The twitter posts are shown on the Live Dashboard now as well.
Issue #8
Modified the publish system to include end publish dates/times and to have a status field in the db. Cron handles updating the status.
Issue #10
Added child_class to the {menu} system to designate other classes other than children to the ul of the child
Issue #4
Added copy function to module -> publish -> admincp.php in order to allow duplicating content easily. Added Copy links for content.php, content_non_standard.php, content_standard.php
Issue #3
Added themed 404 pages using route['404_override'] to point to the module error/view. Then that loads up the template error.thtml.