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

Recaptcha Module and tweaked forms/form_model #14

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

Conversation

josev814
Copy link

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.

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
…t, and added options for bit.ly login and user_key
…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>
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
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant