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

Minor fixes #274

Open
4 of 6 tasks
farzindev opened this issue Apr 21, 2021 · 10 comments
Open
4 of 6 tasks

Minor fixes #274

farzindev opened this issue Apr 21, 2021 · 10 comments
Assignees
Labels

Comments

@farzindev
Copy link
Member

farzindev commented Apr 21, 2021

  • 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.

@fullreboot
Copy link
Contributor

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

#295

@farzindev
Copy link
Member Author

@fullreboot Thank you, we review it this week.

@fullreboot
Copy link
Contributor

fullreboot commented Sep 13, 2021

@farzindev

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?

@fullreboot
Copy link
Contributor

fullreboot commented Sep 14, 2021

The bug previously mentioned has been fixed and is pushed to the PR. If this is an unacceptable hack please let me know.

#295 (comment)

@farzindev
Copy link
Member Author

Hey @rxtur, Please review this, Thank you.

@chatay
Copy link

chatay commented Oct 18, 2021

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?

@fullreboot
Copy link
Contributor

@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

@farzindev
Copy link
Member Author

@chatay it seems it's working fine.

@oleksiivasylenko
Copy link

Hi, I see several more small issues.

  1. When I logged in into admin panel and switching tabs(dashboard/blog/pages...) after once clicking on 'pages'
    the tooltip icon is always show under 'pages' and 'blog' tabs until refresh page.

  2. Create new 'page', save it, then publish it and go to that page. On that page cover image is broken 'src="//http://localhost:3169/img/cover.jpg"'

@ismaelmiah
Copy link
Contributor

@farzindev the default cover image is showing, but this picture should change after clicking on it. I think this feature has bugs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Blogifier v3.0
  
In progress
Development

No branches or pull requests

6 participants