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

bump to 2.8.0 #647

Merged
merged 7 commits into from
Jan 26, 2022
Merged

bump to 2.8.0 #647

merged 7 commits into from
Jan 26, 2022

Conversation

akelad
Copy link
Contributor

@akelad akelad commented Jul 22, 2021

No description provided.

@akelad akelad requested a review from indam23 July 22, 2021 11:37
Dockerfile Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
@indam23 indam23 enabled auto-merge August 5, 2021 15:40
@indam23 indam23 self-requested a review August 9, 2021 21:03
@akelad
Copy link
Contributor Author

akelad commented Aug 16, 2021

@melindaloubser1 should this be merged?

@indam23
Copy link
Contributor

indam23 commented Aug 16, 2021

Yes, it should, but it never did because the discourse API calls in the unit tests are failing

we can force merge it and open an issue to look into that separately, since it means the existing version's calls also won't work anyway.

@akelad
Copy link
Contributor Author

akelad commented Aug 16, 2021

I would rather we fix it, if we have the time 😅

@indam23
Copy link
Contributor

indam23 commented Aug 16, 2021

Ok, in that case I can get to it later this week

@indam23 indam23 merged commit ead5e16 into main Jan 26, 2022
hamzaziizzz pushed a commit to hamzaziizzz/rasa-demo that referenced this pull request Aug 10, 2024
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