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

修改自定义页面路径形式 #1174

Closed
ruibaby opened this issue Nov 30, 2020 · 9 comments · Fixed by #1177 or halo-dev/console#270
Closed

修改自定义页面路径形式 #1174

ruibaby opened this issue Nov 30, 2020 · 9 comments · Fixed by #1177 or halo-dev/console#270
Assignees
Labels
kind/support Categorizes issue or PR as a support question.

Comments

@ruibaby
Copy link
Member

ruibaby commented Nov 30, 2020

目前有不少人反应自定义页面带了 /s/ 二级路径看着很奇怪,所以这个 issue 征集大家的意见。

  1. 去除 /s/,直接使用 slug 根路径,类似 https://ryanc.cc/about
  2. 保留现状,类似 https://ryanc.cc/s/about。 (目前这种方式的 /s/ 二级路径是可以在后台修改的)

image(自己在后台选择路径方式)

@ruibaby ruibaby added the kind/support Categorizes issue or PR as a support question. label Nov 30, 2020
@ruibaby ruibaby pinned this issue Nov 30, 2020
@LIlGG
Copy link
Member

LIlGG commented Dec 1, 2020

个人是比较喜欢直接使用 slug 根路径的方式,也许是强迫症的原因,看到有个多余的二级 URI 就感觉很不舒服。
但以目前的情况,想切换成第一种方式还是有以下问题的:

  1. 旧用户已经被收录的自定义页面将在更新版本后失效。
  2. 旧用户已经存在的自定义页面别名与现有的默认菜单名冲突,将会造成不可预知的问题。
  3. 在新增自定义时,将要校验自定义页面别名是否与现有默认页面名冲突,即需要多查一次库。

如果能解决掉如上所述的问题 ,就建议第一或者 【我全都要???】

@xzhuz
Copy link
Contributor

xzhuz commented Dec 1, 2020

没有用过后台的自定义路径,所以不确定自定义页面路径是否支持自定义成https://ryanc.cc/about?如果可以支持自定义这种,那么感觉就没有必要讨论。即使不支持, 支持自定义页面路径,也可以让使用者自己DIY。

@ruibaby
Copy link
Member Author

ruibaby commented Dec 1, 2020

没有用过后台的自定义路径,所以不确定自定义页面路径是否支持自定义成https://ryanc.cc/about?如果可以支持自定义这种,那么感觉就没有必要讨论。即使不支持, 支持自定义页面路径,也可以让使用者自己DIY。

不支持,必须有二级。

@xzhuz
Copy link
Contributor

xzhuz commented Dec 1, 2020

没有用过后台的自定义路径,所以不确定自定义页面路径是否支持自定义成https://ryanc.cc/about?如果可以支持自定义这种,那么感觉就没有必要讨论。即使不支持, 支持自定义页面路径,也可以让使用者自己DIY。

不支持,必须有二级。

那如果把这个限制放开,只限制路径不能重复。至于需要怎么样的路径,让使用者自己DIY。个人觉得,这样就很好了。

@ruibaby
Copy link
Member Author

ruibaby commented Dec 1, 2020

那如果把这个限制放开,只限制路径不能重复。至于需要怎么样的路径,让使用者自己DIY。个人觉得,这样就很好了。

个人觉得也没必要限制路径不能和内置页面的路径重复,把自定义的优先级提高就好了,比如 /links,我想覆盖这个页面就可以新建一个自定义页面,slug 命名为 links。

@xzhuz
Copy link
Contributor

xzhuz commented Dec 1, 2020

那如果把这个限制放开,只限制路径不能重复。至于需要怎么样的路径,让使用者自己DIY。个人觉得,这样就很好了。

个人觉得也没必要限制路径不能和内置页面的路径重复,把自定义的优先级提高就好了,比如 /links,我想覆盖这个页面就可以新建一个自定义页面,slug 命名为 links。

👍这样设计感觉更好了

@praming
Copy link

praming commented Dec 1, 2020

1不是不好实现吗?如果能实现那肯定选1或者3啊。

@ruibaby
Copy link
Member Author

ruibaby commented Dec 1, 2020

1不是不好实现吗?如果能实现那肯定选1或者3啊。

以前不好实现,但现在好实现了,属于历史遗留问题,但要考虑老用户的兼容问题。

@praming
Copy link

praming commented Dec 1, 2020

以前不好实现,但现在好实现了,属于历史遗留问题,但要考虑老用户的兼容问题。

如果是这样的话我觉得3比较合适啊。

@ruibaby ruibaby changed the title 关于自定义页面路径的意见征集。 修改自定义页面路径形式 Dec 1, 2020
@ruibaby ruibaby self-assigned this Dec 1, 2020
ruibaby added a commit to ruibaby/halo that referenced this issue Dec 1, 2020
@ruibaby ruibaby mentioned this issue Dec 1, 2020
ruibaby added a commit to ruibaby/halo-console that referenced this issue Dec 1, 2020
ruibaby added a commit that referenced this issue Dec 2, 2020
ruibaby added a commit to halo-dev/console that referenced this issue Dec 2, 2020
@ruibaby ruibaby unpinned this issue Dec 3, 2020
XuCpeng added a commit to XuCpeng/halo that referenced this issue Apr 17, 2021
* Create SECURITY.md (halo-dev#1144)

* doc: update swagger contact email. (halo-dev#1147)

* Update README.md

* feat: halo-dev#1103 (halo-dev#1173)

* feat: halo-dev#1174 (halo-dev#1177)

* feat: halo-dev#1158 (halo-dev#1176)

* Upgrade dependencies (halo-dev#1184)

* Update spring boot version and source compatibility version

* Rearrange some configs

* Upgrade swagger to 3.0.0

* Make swagger configurable

* Make swagger-ui.html path backward compatible

* Change jdk version into 11 in ci/cd scripts and Dockerfile

* chore: remove redis cache store (halo-dev#1190)

* chore: remove redis cache store.

* chore: remove redis cache store.

* chore: update build.gradle.

* Make more friendly error track (halo-dev#1191)

* Update gradle wrapper version to 6.6.1

* Upgrade h2 version to 1.4.197

* Make controller log more details

* Refactor FileHandler

* Fix image reader error

* fix: halo-dev#1200 (halo-dev#1207)

* feat: halo-dev#1160. (halo-dev#1203)

* fix: halo-dev#1180 (halo-dev#1209)

* feat: halo-dev#766 (halo-dev#1210)

* feat: add api for markdown export (halo-dev#1199)

* add API for markdown-export

* add front-matter support

* optimize fileName for markdown-export

* fornt-matter与正文中间增加换行符

* Replace travis ci with github action (halo-dev#1212)

* 1196 refactor/ci (#3)

* Remove .travis.yml

* Refactor github action partially

* Fix yaml syntax error

* Add run command for every step

* Set current branch name into halo.yml temporarily

* Test validation.yml

* Add upload-release-asset step into release.yml

* Perfect release.yml

* Fix indent error

* Refactor on condition in release.yml

* Refactor on condition in validation.yml

* Fix release.yml

* Fix upload_url value set

* Fix environment set error

* Change artifact variable from output into global environment

* Fix deprecated environment set method

* Fix environment variable set error

* Change assert_content_type with application/zip

* Refactor upload release step

* Fix release id set

* Fix release id set again

* Fix syntax error

* Refactor upload process

* Refactor halo ci

* Make build step rely on check step

* Inspect docker action

* Inspect docker action again

* Refine bootBuildImage config

* Refactor bootBuildImage config and halo ci

* Fix download artifact path error

* Fix docker image name concat error

* Remove downloaded files inspect tips

* perft: Reduce the scope of pointcut to make the package scope of tangent point as small as possible (halo-dev#1238)

* feat: support custom post password template. (halo-dev#1236)

* pref: clean unnecessary code. (halo-dev#1237)

* pref: clean unnecessary code.

* chore: change org.jetbrains.annotations.NotNull to org.springframework.lang.NonNull.

* Refactor checkstyle (halo-dev#1241)

* Refactor checkstyle.xml and add .editorconfig

* Optimized imports

* Rearrange codes

* Fix check sytle error in source codes

* Reformat test codes

* Fix check style error in test codes

* Config checkstyle plugin

* Fix merge conflicts

* fix: halo-dev#1214. (halo-dev#1242)

* release: 1.4.3-beta.1.

* release: 1.4.3-beta.2.

* Update README.md

* Fix theme updation error (halo-dev#1217)

* Make rest controller loggable

* Refactor pull from git process

* Replace Callback interface with Consumer

* Tag theme fetch apis and services deprecated

* Add getAllBranchesTest

* Refactor theme fetcher partially

* Refactor theme property scanner

* Add ThemeFetcherComposite

* Add InputStreamThemeFetcher

* Accomplish multipart zip file theme fetcher

* Reformat ThemeServiceImpl

* Reformat codes

* Provide ThemeRepository

* Complete MultipartFileThemeUpdater

* Make CommonsMultipartResolver support put request method

* Replace some methods with ThemeRepository

* Add GitThemeUpdater

* Add merge two local repo test

* Refine merge process with two repos

* Add more test entry point in GitTest

* Add shutdown hook after creating temporary directory

* Add test: find commit by tag

* Refactor git clone process in GitThemeFetcher

* Refine merge process of two repo

* Make sure that RevWalk closed

* Fix FileUtils#findRootPath bug

* Add clean task before gradle check

* Add fallback theme fetcher

* Disable logback-test.xml

* Set testLogging.showStandardStreams with true

* Fix test error while missing halo-test folder

* Enhance git theme fetcher

* Add copy hidden folder test

* Refine GitThemeFetcherTest

* Accomplish GitThemeUpdater

* Accomplish theme update

* Fix checkstyle error

* Add more deprecated details

* Refactor Dockerfile with layered jar (halo-dev#1248)

* Refactor Dockerfile with layered jar

* Add projectVersion print task

* Refactor docker build process

* Remove aliyun maven mirror

* Correct multi platforms list

* Correct multi platforms list again

* Make docker platforms configurable

* Fix folder copy error: ProviderMismatchException (halo-dev#1249)

* fix: unable to access custom sheet. (halo-dev#1246)

* release: 1.4.3-beta.2.

* doc: add document website. (halo-dev#1215)

* doc: add document website.

* doc: add document website.

* doc: add document website.

* release: 1.4.3.

* Update README.md

* Update README.md

* feat: halo-dev#1225 (halo-dev#1235)

* feat: halo-dev#1225

* fix: compatible with jdk1.8

* fix: format code

* fix: fix the problem that the status of the recycle bin file is incorrect when revocering

* fix: format code

* fix: format code

* fix: fix the post cannot be converted to recyling mode

* fix: post cannot be published on deleting the category password

* fix: fix jpa error

* fix: format code

* fix: encryption type extracted into enum

* fix: format code

* fix: format code

* fix: changes requested

* fix: format code

* fix: revert checkstyle.xml

* fix: change request

* fix: not encrypt 方法改为重载方法

* fix: 修复因调整 git 版本被回退的代码

Co-authored-by: xiangbei.yzx <[email protected]>

* fix: halo-dev#1255. (halo-dev#1256)

* fix: halo-dev#1255.

* fix: halo-dev#1255.

* release: 1.4.4.

* fix: category authentication redirect url. (halo-dev#1264)

* pref: journals likes api. (halo-dev#1266)

* pref: sitemap.xml (halo-dev#1267)

* release: 1.4.5.

* pref: grouping assertions (halo-dev#1273)

* Grouping assertions

Signed-off-by: Elvys Soares <[email protected]>

* Grouping assertions

Signed-off-by: Elvys Soares <[email protected]>

* Grouping assertions

Signed-off-by: Elvys Soares <[email protected]>

* fix: code style check.

Co-authored-by: Ryan Wang <[email protected]>

* Fix Page response model inconsistent in swagger ui (halo-dev#1277)

* Provide backup dto fetch api (halo-dev#1278)

* Fix swagger security reference config error

* Add backup dto fetch api

* Rearrange fetch api

* Fix incorrect cache lock of journal like api (halo-dev#1279)

* fix: fetch work dir backup api. (halo-dev#1282)

* feat: support import user. (halo-dev#1283)

* fix: theme update by upload. (halo-dev#1284)

* Cache current theme in theme repository (halo-dev#1286)

* pref: halo-dev#1050 (halo-dev#1287)

* Update FUNDING.yml

* pref: init default theme. (halo-dev#1297)

* feat: 扩展 freemarker 实现 block (halo-dev#1295)

* fix halo-dev#950 附件不存在时删除报错的问题

* pref: 扩展 freemarker 增加 block 功能

close halo-dev#1292

* checkStyle

* add unit test

* update test

* Upgrade spring boot version (halo-dev#1289)

* Update gradle wrapper version

* Update spring boot version to 2.5.0-M2

* Fix wrong const of temp_dir

* Refactor error controller

* Fix startup error due to theme not found

* Refine error controller handler

* Refine multipart resolver config

* Fix ThemeRepositoryImplTest error

* Fix freemarker not found error

* chore: change jetty to undertow.

* Remove useless throws

Co-authored-by: Ryan Wang <[email protected]>

* Make more tolerant of reading image (halo-dev#1298)

* Fix error for updating activated theme (halo-dev#1300)

* Fix error about response committed (halo-dev#1301)

* Add index page request test

* Add test for first page request

* Create session before requesting content

* chore: sync default theme. (halo-dev#1304)

* Refactor system configuration (halo-dev#1303)

* Refactor application*.yml

* Remove application-user.yaml

* Fix invalid config for aspect

* Remove random theme folder generation (halo-dev#1305)

* release: 1.4.6.

* feat: halo-dev#1313 (halo-dev#1315)

* release: 1.4.7-beta.1

* release: 1.4.7

* feat: add index page to sitemap.xml (halo-dev#1318)

* style: set the color of log to default after banner (halo-dev#1321)

* Upgrade to Spring Boot 2.5.0-M3 (halo-dev#1324)

* feat: enable swagger configuration in development. (halo-dev#1328)

* Automate attribute converter (halo-dev#1325)

* Deprecate AbstractConverter

* Remove unused enum and attribute converter

* Add AttributeConverterApplyTest

* Add JpaConfiguration

* Add AttributeConverterAutoGenerator

* Integrate automate-attribute-converter

* Rename JpaConfiguration

* Remove useless attribute converters

* Exclude property enums for auto-generating

* Refine JournalType definition

* Fix an error about existing injected type

* fix: halo-dev#1311 (halo-dev#1327)

* feat: halo-dev#1036 (halo-dev#1331)

* feat: halo-dev#1036

* revert: BaseCommentRepository.

* fix: incorrect number of children comment. (halo-dev#1332)

* feat: add gravatar source field for comment options api. (halo-dev#1333)

* chore: remove deprecated code. (halo-dev#1334)

* chore(deps): upgrade ali oss sdk dependency. (halo-dev#1335)

* revert: halo-dev#1297 (halo-dev#1343)

* pref: RSS/Atom (halo-dev#1342)

* 1.Add the lastBuildDate in RSS.
2.Add the updated in Atom.
3.Change the date format in RSS and Atom
4.Add the lastModified in the response header.

* fix code style

* Add OptionFilter for bulk option api (halo-dev#1345)

* Add OptionFilter for bulk option api

* Add another filter method for single option

* Restrict OptionController response

* Remove redundant api

* feat: complete private option keys.

* feat: complete private option keys.

Co-authored-by: Ryan Wang <[email protected]>

* release: 1.4.8

* doc: update readme.

* doc: update readme.

* fix: Returns the result in the content api for options getby (halo-dev#1353)

Co-authored-by: GalvinGao <[email protected]>
Co-authored-by: Ryan Wang <[email protected]>
Co-authored-by: John Niang <[email protected]>
Co-authored-by: Raremaa <[email protected]>
Co-authored-by: guqing <[email protected]>
Co-authored-by: zhixiangyuan <[email protected]>
Co-authored-by: xiangbei.yzx <[email protected]>
Co-authored-by: Elvys Soares <[email protected]>
Co-authored-by: Li <[email protected]>
Co-authored-by: 知雨 <[email protected]>
Co-authored-by: 肥鱼先生 <[email protected]>
Co-authored-by: Lay <[email protected]>
Co-authored-by: 扶醉 <[email protected]>
ruibaby added a commit to ruibaby/halo that referenced this issue Jun 11, 2021
JohnNiang pushed a commit to JohnNiang/halo that referenced this issue Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/support Categorizes issue or PR as a support question.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants