-
Notifications
You must be signed in to change notification settings - Fork 55
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
Upgrade to MVC5, new themes, refresh all packages #55
base: master
Are you sure you want to change the base?
Conversation
…till stores roles but they are mapped to claims in the app which does not use roles at all.
…shown from the db.
There was an invalid reference to the MVC dll in the test project.
In the WikiController's test class the constructor missed a SettingsProvider parameter.
Admin header width changed to dynamic (previously it was 300px)
There is a condition where we check the DatabaseIssue property, but it was check to be true where false is the proper value (in Login.cshtml)
Miniprofiler caused login page rendered in footer
New post field capitaization and displayname for selected tags
Fixed the blog entry width
ShowDown and WMD is replaced by PageDown (v2014-10-10)
Optical tuning of the new/edit post page
Fixed the radiobutton formatting
Updated the date to the standard HTML5 time tag
Added the HTML5 article and header tags for better SEO
Added bootstrap glyphs and modified the basic theme to show them before and/or after the post summary
Fixed all links to images in themes
Upgraded jQuery to 2.1.3
Updated Disqus to use the web API and changed the way we get the number of comments from Disqus. The display of the comment count was changed so we have a uniform CSS styling for DB comments, Disqus comments and when we have the comments disabled. We have now a new field on the admin page to enter our Disqus API Key
The theme-engine was reworked to use WordPress themes. There are only 2 themes which look fine: hostmarks and quark.
There are now 4 WordPress themes which can be applied to FunnelWeb. The old themes were removed.
Admin area was redesigned by applying a bootstrap theme on it. Default CSS files were striped, the whole theme handling is clearer, simpler then before. Bootstrap 3.3.2 is also included now.
Minor design fixes in the admin area
Fixed the new grid system's CSS in a few pages
Small fixes around the themes, especially in the commenting area.
Markdown preview is now fixed on the comment partial view and the admin/edit page too.
Tag editor is replaced with bootstrap tag editor, but the functionality is disabled until the DB access layer is replaced by Entity Framework 6.
EnforcedString and PageName class explanation was added.
SQL CE database is included now. Also fixed a script error which prevented creating the initial sql ce database. Added a script to add the WordPress default entries on initialization.
Added a few screenshots of 4 wordpress themes to the docs folder
Fixed a small issue in flower theme
Updated version info to 2.5
Added some info related to the updated version.
0_o I think I should just add you and @noopman as colaborators of funnelweb? |
Not that this isn't welcome, I am just not using FunnelWeb at the moment and you guys will likely do it justice helping maintain it? |
@andrasfuchs This is major! I see you have been busy! I will get these changes and see if I can integrate them with my blog branch to test it out. |
Thanks! @JakeGinnivan Jake, sure, you can add me to the collaborators. This version needs to be tested and fixed here and there, but I hope others can join us! @janierdavila Obviously I've made some breaking changes too, so please be careful with it :) I know that a few things are not working well, like the tagging control, sometimes I've got an NHibernate error while saving the settings etc., and I didn't have the time to test all the features (like Azure authentication). On the other hand, I would be glad to get the some feedback or even fixes ;) |
@andrasfuchs you have push access. You could push this to a branch in the main repo and get others to help out while it is stabilised |
@JakeGinnivan Great, thank you! I probably missed something, but I was unable to create a new branch in FunnelWeb. Could you double-check my settings or could you create a "mvc5-beta" branch for us? |
@andrasfuchs Have you tried this from your git console?:
|
@janierdavila I'm clearly not a Git pro... Using your suggested According to this StackOverflow entry, you are right, that's the way to go: first I need to create a new branch in my (head) repo and then create a new pull request and then push it into a new branch at the base repo. BUT, this is what I get when I'm trying to pull it into the master (looks OK): and this is when I enter the "mvc5-beta" as the destination branch: |
@andrasfuchs Since you have push access now, I was thinking you could just push that new branch to this repo instead of your fork. You can try these steps to add this repo to your list of remotes and push directly.
Let me know if that works |
Hey @andrasfuchs What you want is to have two remotes, upstream and origin. Or base as @andrasfuchs, I use upstream but either way origin should point to your fork. To get yourself where you need to be do this
A mvc5-beta branch will then exist in the main repo. You don't do this via a pull request, you push with the last command above. Once it is in the main repo you can close this PR and open a new one between the |
I think you just need the last command tbh, but resetting your master to not have this stuff on it is a good idea as it will stop accidental pushes of this code into master before you are ready |
Thanks for the tips guys! My default upstream is not the original FunnelWeb repo, but noopman's fork. I had to add the original repo as a named remote, but it seemed to work. When I tried the Nonetheless I continued to run the @JakeGinnivan could you double-check my access rights? |
You need to do a I added you to a team, but didn't add the team to the repo. You should have access now |
Looks good, the source is in the mvc5-beta branch now, thanks! |
@janierdavila I disabled MiniProfiler (in _Public.cshtml and _Private.cshtml) when I started to work with FunnelWeb, I'm not sure if it works now if you put it back. |
New features in 2.5 (2015-03-30)
Screenshots
Admin
WP-Flower theme
WP-Hostmarks theme
WP-Quark theme
WP-Topmag theme
Further Information
If you need some help with .NET, C#, MVC, Entity Framework, HTML5, CSS, JavaScript, jQuery or Bootstrap, you can contact me for one-on-one consultancy or live training on the Kevin Sharp Traning website.