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.

Sign up for GitHub

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

Jump to bottom

apollo的前端提交页面有增加diff渲染的计划吗？ #4227

Open
coderpolo opened this issue Feb 15, 2022 · 10 comments
Open

apollo的前端提交页面有增加diff渲染的计划吗？ #4227

coderpolo opened this issue Feb 15, 2022 · 10 comments
Labels
area/portal apollo-portal area/ui Categorizes issue or PR as related to front end feature request Categorizes issue as related to a new feature.

Comments

@coderpolo
Copy link

coderpolo commented Feb 15, 2022 •
edited
Loading

在提交页面，左右两列做diff。后面会有这个feature吗？
因为有时候单个key的value是一个长字符串
@nobodyiam
Copy link
Member

Would you please share more detail on this?

@coderpolo
Copy link
Author

coderpolo commented Feb 17, 2022 •
edited
Loading

apollo-diff

如上图：对指定key编辑之后，在新的value提交前，对新旧value做一个diff渲染，这样就可以明确看到本次更改的内容呀

@nobodyiam nobodyiam added the feature request Categorizes issue as related to a new feature. label Feb 18, 2022
@nobodyiam
Copy link
Member

Currently, there is a diff page when releasing the configurations. I'm not sure whether it's a common requirement to show the diff page when the user is editing the configuration item.

image

@coderpolo
Copy link
Author

对，想要的就是这样的diff功能。目前看来已经满足了，请问上面图片对应的apollo版本是？ 我们可能需要升级一下了

@coderpolo
Copy link
Author

image
我本地用quick-start-1.9.0启动的实例：在发布前的页面是这样的，没有diff渲染额

@nobodyiam
Copy link
Member

This is the release form for non-properties namespaces, e.g. yaml

http://106.54.227.205/config.html?#/appid=100004458

image

@coderpolo
Copy link
Author

您好，非常期望properties格式的配置也具备diff渲染功能

@Anilople Anilople added area/portal apollo-portal area/ui Categorizes issue or PR as related to front end labels Feb 26, 2022
@coderpolo
Copy link
Author

coderpolo commented Feb 22, 2023 •
edited
Loading

我又回来了，这个计划现在怎么样啦？ @nobodyiam

@muqidi
Copy link

muqidi commented May 18, 2023

mark

1 similar comment
@jarvislin94
Copy link

mark

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/portal apollo-portal area/ui Categorizes issue or PR as related to front end feature request Categorizes issue as related to a new feature.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@nobodyiam @jarvislin94 @muqidi @Anilople @coderpolo