Skip to content

Conversation

yorunning
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Apr 7, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @Yidadaa on Vercel.

@Yidadaa first needs to authorize it.

@Yidadaa
Copy link
Collaborator

Yidadaa commented Apr 7, 2023

为什么不让移动端撤销删除?

@yorunning
Copy link
Contributor Author

yorunning commented Apr 7, 2023

为什么不让移动端撤销删除?

移动端关闭会话时确认框点取消时,也会弹出撤销提示

@Yidadaa
Copy link
Collaborator

Yidadaa commented Apr 7, 2023

那正确的改法应该是把 confirm 的结果提升到变量里来判断,而不是直接在移动端禁用这个交互。

@yorunning
Copy link
Contributor Author

那正确的改法应该是把 confirm 的结果提升到变量里来判断,而不是直接在移动端禁用这个交互。

抱歉,我的表述有问题,这个提交已经修复了,把撤销提示的代码提入到removeSession()同一层级就可以了。

@yorunning yorunning changed the title fix: hide toast on cancel session deletion on mobile fix: hide toast when confirmation box is cancelled Apr 7, 2023
@Yidadaa Yidadaa closed this Apr 7, 2023
@Yidadaa Yidadaa reopened this Apr 7, 2023
@Yidadaa Yidadaa merged commit 9afed21 into ChatGPTNextWeb:main Apr 7, 2023
@yorunning yorunning deleted the fix branch April 13, 2023 00:38
yxl pushed a commit to yxl/ChatGPT-Next-Web that referenced this pull request May 4, 2023
fix: hide toast when confirmation box is cancelled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants