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

Port to CodeIgniter 3.X #5

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

Conversation

hari-govind
Copy link

Hi bro,
As a follow up to #4 , I've made necessary changes that make Treasure-Hunt compatible with the latest stable version of CodeIgniter(3.1.10). I've casually tested various features to make sure nothing is broken, but we will be doing a more through test before using Treasure-Hunt for our event and will fix any bugs that we encounter and make pull requests necessary.
I'll update the readme within next weekend.
Kindly review it and suggest changes if necessary.
Regards

@xaneem
Copy link
Owner

xaneem commented Feb 4, 2019

Thank you so much @hari-govind. This looks good; let's merge it after your event is complete so that it doesn't break for anyone.

Also, do share the link when your treasure hunt event is live :)

@utkarshdubey
Copy link

Even I want to play your cryptic hunt. Post the link here. :)

@hari-govind
Copy link
Author

You're welcome @xaneem :) And that's a good idea to merge after the event so that all commits related to the upgrade can be merged at once.
@utkarshdubey2 thank you very much for your interest, I'll post the link once we have got this hosted so that you both can also play! :)

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