Skip to content

Text Exercise: Integrate the new Athena - #174

Open
terlan98 wants to merge 43 commits into
developfrom
feature/text/new-athena
Open

Text Exercise: Integrate the new Athena#174
terlan98 wants to merge 43 commits into
developfrom
feature/text/new-athena

Conversation

@terlan98

@terlan98 terlan98 commented Aug 14, 2023

Copy link
Copy Markdown
Contributor

⚠️ Do not merge before ls1intum/Artemis#7136

This PR integrates the new Athena implementation for text exercises into Themis.

Can be tested on https://ma-schwind.ase.cit.tum.de/ with Athena Test Course 2 (make sure feature/programming-exercises/feedback-suggestions-server is deployed)

The text blocks with feedback suggestions are indicated with underline:
Screenshot 2023-08-28 at 14 06 33

The user can tap on such blocks and accept or dismiss the suggestion
Screenshot 2023-08-28 at 14 06 33

Suggested feedbacks are shown with a badge in correction sidebar:
Screenshot 2023-10-26 at 21 27 23

@terlan98
terlan98 changed the base branch from develop to bugfix/courseview-grouping August 14, 2023 10:30
@terlan98
terlan98 marked this pull request as ready for review August 30, 2023 17:33
Base automatically changed from bugfix/courseview-grouping to develop September 5, 2023 09:34
@terlan98
terlan98 marked this pull request as draft September 10, 2023 07:13
@terlan98
terlan98 marked this pull request as ready for review September 10, 2023 08:01
@terlan98 terlan98 changed the title Text Exercises: Integrate the new Athena Text Exercise: Integrate the new Athena Oct 10, 2023
Conflicts:
	Themis.xcodeproj/project.pbxproj
@terlan98
terlan98 marked this pull request as draft October 26, 2023 08:30
@terlan98
terlan98 marked this pull request as ready for review October 26, 2023 17:33
@maximiliansoelch

Copy link
Copy Markdown
Member

Should the changes to the docs folder be included in this PR?

@terlan98

Copy link
Copy Markdown
Contributor Author

Should the changes to the docs folder be included in this PR?

No, I'm not sure how they ended up in this PR 😅. I removed them with a new commit.

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