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

perf: persistent prompts and add chat record #540

Merged
merged 47 commits into from
May 30, 2024
Merged

perf: persistent prompts and add chat record #540

merged 47 commits into from
May 30, 2024

Conversation

jamebal
Copy link
Contributor

@jamebal jamebal commented May 15, 2024

  1. Persistent cue word

  2. Add chat log interface

jamebal added 30 commits May 23, 2023 11:23
@jamebal
update actions
9eccfde
@jamebal
Merge branch 'main' into jmal
562b953
@jamebal
chore(gitignore): update ignored files
4830604
@jamebal
feat(*): add chat record
d8276a7
@jamebal
refactor: add chat record
7414dc5
@jamebal
refactor: add chat record
7c0d42e
@jamebal
refactor: add chat record
23e7d2e
@jamebal
perf: chat record
38e03f1
@jamebal
perf: chat record
92a7b61
@jamebal
perf: remove Shift+Enter line breaks from text copied to the clipboard
6e2811f
@jamebal
perf: chat record
485df4d
@jamebal
feat: the prompt word store is saved to the database
5809f22
@jamebal
perf: the prompt word
154df7b
@jamebal
perf: the prompt word
aa5a3d3
@jamebal
feat: persistent prompts
09badfd
@jamebal
Update README.md
263a1b0
@jamebal
fix: the typewriter animation
e52b43b
@jamebal
Merge branch 'main' into test
3256961
@jamebal
Merge branch 'main' into jmal
f87be7b
@jamebal
fix: the typewriter animation
ecf6d4e
@jamebal
Merge remote-tracking branch 'origin/main'
f9016d2 
# Conflicts:
#	service/src/index.ts
#	service/src/storage/model.ts
#	src/api/index.ts
#	src/components/common/Setting/model.ts
@jamebal
Merge branch 'main'
1dd5102
@jamebal
fix: fix the max tokens for gpt-4-turbo 128K
3002891
@jamebal
feat: add custom system role
6800e38
@jamebal
feat: add GeminiPro option
3d20329
@jamebal
feat: add chat record
3771a52 
feat: persistent prompts
feat: add custom system role
feat: add GeminiPro option
@jamebal
fix: chat record
298a4ae
@jamebal
perf: persistent prompts
a9f1ccd
@jamebal
style: update default systemMessage
b3935c0
@jamebal
Merge branch 'refs/heads/master' into jmal
deebf10
jamebal added 8 commits May 14, 2024 15:14
@jamebal
Merge remote-tracking branch 'refs/remotes/origin/main' into jmal
9090954 
# Conflicts:
#	README.md
#	service/src/chatgpt/index.ts
#	service/src/index.ts
#	service/src/storage/config.ts
#	service/src/storage/mongo.ts
#	src/api/index.ts
#	src/components/common/Setting/General.vue
#	src/components/common/Setting/index.vue
#	src/store/modules/user/index.ts
#	src/utils/copy.ts
@jamebal
chore: merge code
6d655d5
@jamebal
build: v2.17.0
46f057c
@jamebal
test: build
129b905
@jamebal
perf: history logs plus model
0533956
@jamebal
Merge remote-tracking branch 'refs/remotes/origin/main' into jmal
cd71aa9
@jamebal
style: remove redundant code
42e6144
@jamebal
chore: restore main branch
3c9bf41
@BobDu
Copy link
Member

BobDu commented May 15, 2024

Sorry, before reviewing the logic changes, can you please merge the upstream main to the feature branch and then run lint?

In addition, please revert modifications to unrelated files such as gitignore and readme.

@jamebal
Copy link
Contributor Author

jamebal commented May 15, 2024

  1. already merge the upstream main branch.
  2. already revert modifications to unrelated files such as gitignore and readme.

@BobDu
Copy link
Member

BobDu commented May 15, 2024

Are you already run pnpm lint:fix?

@jamebal
Copy link
Contributor Author

jamebal commented May 15, 2024

Are you already run pnpm lint:fix?

yes

BobDu
BobDu reviewed May 15, 2024
View reviewed changes
src/api/index.ts Outdated Show resolved Hide resolved
@BobDu
Copy link
Member

BobDu commented May 27, 2024

ping @jamebal
Some comment in code review.

BobDu
BobDu requested changes May 30, 2024
View reviewed changes
Copy link
Member

@BobDu BobDu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you check for duplicate files that might not be used?

src/components/common/Setting/Message/Text.vue Outdated Show resolved Hide resolved
src/components/common/Setting/Message/index.vue Outdated Show resolved Hide resolved
src/components/common/Setting/Message/style.less Outdated Show resolved Hide resolved
src/components/common/Setting/Message/Avatar.vue Outdated Show resolved Hide resolved
@jamebal
perf: chat record
b515ee6 
1.remove redundant files
2.the content of the chat record uses "@/views/chat/components/Message"
BobDu
BobDu approved these changes May 30, 2024
View reviewed changes
Copy link
Member

@BobDu BobDu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BobDu BobDu merged commit c8b1bdd into chatgpt-web-dev:main May 30, 2024
2 checks passed
@zhujunsan
Copy link
Contributor

better add index in mongodb:

db.chat.createIndex({ roomId: 1, dateTime: -1 });

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhujunsan zhujunsan zhujunsan left review comments

@BobDu BobDu BobDu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jamebal @BobDu @zhujunsan