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

[Hack Update] 047-TrafficControlWithDapr- added ASCII GUI into simulation #525

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vjirovsky-msft
Copy link
Contributor

this PR changes output in Simulation console app from plain text to graphic interface (built on top of ASCII art) for better understanding

@jordanbean-msft

@vjirovsky-msft vjirovsky-msft changed the title dapr wth #47 - added ASCII GUI into simulation [Hack Update] 047 TrafficControlWithDapr- added ASCII GUI into simulation Nov 14, 2022
@vjirovsky-msft vjirovsky-msft changed the title [Hack Update] 047 TrafficControlWithDapr- added ASCII GUI into simulation [Hack Update] 047-TrafficControlWithDapr- added ASCII GUI into simulation Nov 14, 2022
@jrzyshr jrzyshr self-assigned this Aug 29, 2023
@vjirovsky-msft
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@perktime perktime self-requested a review September 26, 2023 19:31
@perktime perktime requested a review from Whowong September 26, 2023 19:35
@jordanbean-msft
Copy link
Contributor

jordanbean-msft commented Oct 10, 2023

@vjirovsky-msft, I'm sorry for the terrible delay in getting to this item, I have updated a branch in my fork that fixes the merge conflicts since you submitted this PR, please pull my changes, retest, and update your branch with these changes.

https://github.com/jordanbean-msft/WhatTheHack/tree/feature/dapr-ascii-gui

Use the code below to pull my changes from my fork into yours (update the names & branches)

git remote add theirusername [email protected]:theirusername/reponame.git
git fetch theirusername
git checkout -b mynamefortheirbranch theirusername/theirbranch

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.

4 participants