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

Releases/2.1.0 #215

Merged
merged 128 commits into from
Oct 14, 2023
Merged

Releases/2.1.0 #215

merged 128 commits into from
Oct 14, 2023

Conversation

lakhoune
Copy link
Member

@lakhoune lakhoune commented Sep 20, 2023 •
edited
Loading

This PR includes the following new features:

  • Adds logging that is conform with the XES standard. These logs can be used to apply process mining for bots

Samuel Kwong and others added 30 commits July 4, 2023 16:18
create branch
777ee79
moved code from openai_wrapper to here
2b267ec
messy code but a simple completion request can be made to openAI API,…
350d804 
… need credits to work with response and still need to format the response
Removed api key, was automaticlaly detected and removed by openAi, wi…
eaa3231 
…ll find solution to safely use key
@lakhoune
add branch to ci
5ce08c0
@lakhoune
close #200
a678c06
@lakhoune
autoincrement on int
f6ca672
openai_api_key as env variable
ec83be9
@lakhoune
format
f82971c
@lakhoune
implement getCustomMessageDescriptions
4afaa73
@lakhoune
move intent determination to seperate function
604948b
@lakhoune
remove context
2279b93
@lakhoune
adds rasa to docker env and updates env vars
439addc
@lakhoune
add test stage
91b4cfc
@lakhoune
fixes error in test compilation
33da041
@lakhoune
wait for test
62bca03
@lakhoune
add init script
e19ae0e
@lakhoune
add sbm properties file to gitignore
8867fdf
@lakhoune
add data processing to sbm
d427a61
@lakhoune
bump l2p
2f1524a
@lakhoune
fix classpath entry for main package
3c714d9
@lakhoune
store the conversation id with the incoming message
05cf9ad
@lakhoune
wip
3710e0a
@lakhoune
wip
7b679e6
@lakhoune
wip
0040705
@lakhoune
use snapshot
26b9327
@lakhoune
update monitoring messages
cd10e2f
@lakhoune
fix missing constructor
ef23144
@lakhoune
update method name
1c20a22
@lakhoune
update env var
27bcd58
samuelvkwong and others added 9 commits September 22, 2023 16:26
@samuelvkwong
Merge pull request #216 from rwth-acis/sequentialprocess
a367769 
Sequentialprocess
fix merge conflicts from release/1.7.0
b62ac02
@samuelvkwong
Merge pull request #217 from rwth-acis/sequentialprocess
cca7ff3 
fix merge conflicts from release/1.7.0
@lakhoune
image env
fae6aec
@lakhoune
fix default value for webconnector
5aaf7b8
bot parser added missing trigger from ba to ba, also added missing bo…
460796a 
…dy fields to ba trigger another ba
@samuelvkwong
Merge pull request #218 from rwth-acis/sequentialprocess
bf38aac 
bot parser added missing trigger from ba to ba, also added missing bo…
added all leadsTo messages to the followup messages of an incoming me…
a675e86 
…ssage if a bot action succeeds
@samuelvkwong
Merge pull request #219 from rwth-acis/sequentialprocess
abfaf35 
added all leadsTo messages to the followup messages of an incoming me…
@lakhoune lakhoune linked an issue Sep 25, 2023 that may be closed by this pull request
[BUG] State is advanced for no apparent reason #220
Closed
@lakhoune lakhoune marked this pull request as ready for review October 5, 2023 17:53
@lakhoune lakhoune changed the title Releases/1.7.0 Releases/2.1.0 Oct 8, 2023
@lakhoune
Copy link
Member Author

lakhoune commented Oct 8, 2023

Ready to be merged.
I noticed that the latest version is 2.0.2 so i changed the service version 2.1.0

@samuelvkwong
Copy link
Contributor

I finished testing on dev. When a bo action triggers an OpenAPI type bot action, the webconnectorUrl needed to be set as the service endpoint so I added that. There was also a type on the new version number so I fixed that. It is ready to be merged into main on my part.

AlexanderNeumann
AlexanderNeumann requested changes Oct 11, 2023
View reviewed changes
Copy link
Member

@AlexanderNeumann AlexanderNeumann left a comment

Choose a reason for hiding this comment

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

It looks good for me. One minor thing: could you fix the readme icon? =)

@lakhoune lakhoune merged commit 0b28d47 into master Oct 14, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexanderNeumann AlexanderNeumann Awaiting requested review from AlexanderNeumann

Assignees

@lakhoune lakhoune

@samuelvkwong samuelvkwong

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] State is advanced for no apparent reason
3 participants
@lakhoune @samuelvkwong @AlexanderNeumann