Skip to content
This repository was archived by the owner on Aug 18, 2020. It is now read-only.

Remove MockupChat #30

Closed
sebinside opened this issue Jun 17, 2019 · 1 comment
Closed

Remove MockupChat #30

sebinside opened this issue Jun 17, 2019 · 1 comment
Assignees
@sebinside
Labels
bug Something isn't working minor Quick to implement
Milestone
pre-alpha 3

Comments

@sebinside
Copy link
Member

Right now, the MockupChatConnector and Input are completely wrecked. The code was copied from an way earlier version of chat overflow and lack at every point.

A completely new implementation for the new framework is needed, only the parsing might be the same. Also MockupChatOutput etc. should be added.

See: https://github.com/codeoverflow-org/chatoverflow/blob/4e40b6d12f91f24b7d346f993c17a2f8e438d657/src/main/scala/org/codeoverflow/chatoverflow/requirement/service/mockup/MockUpChatConnector.scala
@sebinside sebinside added enhancement New feature or request major Needs a major platform / framework / build environment rework labels Jun 17, 2019
@sebinside sebinside added this to the pre-alpha 3 milestone Jun 17, 2019
@sebinside sebinside added the future This issue is not relevant for the current development, but will become important in the future label Jun 17, 2019
@sebinside sebinside added bug Something isn't working minor Quick to implement and removed enhancement New feature or request future This issue is not relevant for the current development, but will become important in the future major Needs a major platform / framework / build environment rework labels Aug 7, 2019
@sebinside sebinside self-assigned this Aug 7, 2019
@sebinside sebinside changed the title Rework MockupChat Remove MockupChat Aug 7, 2019
@sebinside
Copy link
Member Author

Right now, the mockup chat is just legacy garbge. should be restarted from scratch, the current version deleted.

promasu added a commit that referenced this issue Aug 16, 2019
@promasu
Remove MockUp Chat Connector closes #30
508ea8b 
Signed-off-by: Adrian Nöthlich <git@promasu.tech>
@stewue stewue mentioned this issue Aug 16, 2019
Merged
sebinside added a commit that referenced this issue Aug 16, 2019
@sebinside
Merge pull request #119 from codeoverflow-org/feature/30-remove-mocku…
054001c 
…pchat

Remove MockUp Chat Connector closes #30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sebinside sebinside

Labels
bug Something isn't working minor Quick to implement
Projects
None yet
Milestone
pre-alpha 3
Development

No branches or pull requests
1 participant
@sebinside