-
Notifications
You must be signed in to change notification settings - Fork 518
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
Minor fixes #274
Comments
Alright sorry about all the mess, this pull request will fix # 3 and # 6 from the list above. Had to do some small hackery to make sure we weren't hijacking the /admin route from razor |
@fullreboot Thank you, we review it this week. |
I found another small bug in my pull request. For some reason routing of /admin or /admin/ gets routed differently. Sometimes it get "stolen" by MVC home controller. Maybe you guys have a better idea of why routing is broken? |
The bug previously mentioned has been fixed and is pushed to the PR. If this is an unacceptable hack please let me know. |
Hey @rxtur, Please review this, Thank you. |
hello @farzindev, I am going to look at the second issue. You mean, when one enters the website, register page should be the first initial page appears? @fullreboot 3. and 6. issues have been resolved? |
@chatay 3 and 6 are resolved. 2 doesn't seem to be an issue for me, it will correctly redirect me to register if DB is missing on app startup |
@chatay it seems it's working fine. |
Hi, I see several more small issues.
|
@farzindev the default cover image is showing, but this picture should change after clicking on it. I think this feature has bugs. |
Return URL
Return URL is not working, After we log in, It goes back to the dashboard.
http://localhost:5000/admin/login/?returnUrl=http://localhost:5000/admin/profile/
Register
When we start fresh it goes to the /admin/ instead of /admin/register/
Page URL
There shouldn't be a /posts/ before the slug of the page. It should be:
http://example.com/title-of-the-page
Default Cover
Add the default cover back to the settings,
so that admin can change the default cover.
Localizer on the themes
not working, please check.
Search bug
When we search in the blog posts, we see results from pages as well. they need to be separated completely.
..
Add more minor details that are left to do here.
The text was updated successfully, but these errors were encountered: