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

Add plot cloning #228

Merged
merged 20 commits into from Jan 10, 2020
Merged

Add plot cloning #228

merged 20 commits into from Jan 10, 2020

Conversation

@jasonwynn10
Copy link
Owner

jasonwynn10 commented Aug 22, 2019

  • This PR actually submits something - don't use this system as a messaging service!
  • This PR isn't duplicated - you can check if it is by scanning the small list - link is on the navigation bar for this repository.
  • This PR includes appropriate markdown for sections - e.g. code blocks for suggested code.
  • This PR description and comments in code is understandable - feel free to use your native language to write if you are not comfortable with English.
  • This PR has a single branch for itself in your fork - don't just commit to the master branch of your repository!
  • This PR has comments written in clear English - please don't write unreadable comments just for your eyes.

Pull Request description

This PR adds and implements the /p clone command.

Optional information

English translation is added by default, but other translations are needed.

@Aericio

This comment has been minimized.

Copy link
Collaborator

Aericio commented Aug 22, 2019

"Clone" sounds better for this, in my opinion.

@jasonwynn10 jasonwynn10 changed the title Add plot copy/paste Add plot cloning Aug 22, 2019
jasonwynn10 added 2 commits Aug 22, 2019
@Wertzui123

This comment has been minimized.

Copy link
Contributor

Wertzui123 commented Dec 6, 2019

Will this be ever merged 🤔

jasonwynn10 and others added 6 commits Jan 3, 2020
# Conflicts:
#	resources/chs.ini
#	resources/cze.ini
#	resources/deu.ini
#	resources/fre.ini
#	resources/kor.ini
#	resources/nld.ini
#	resources/spa.ini
#	resources/tur.ini
#	resources/vie.ini
#	resources/zho.ini
@jasonwynn10 jasonwynn10 merged commit ffd0f60 into master Jan 10, 2020
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/push The Travis CI build failed
Details
poggit-ci/MyPlot Created Dev build #499 (&69039): Lint passed
Details
poggit-ci/myplot Created PR build #114 (&69040): Lint passed
Details
@jasonwynn10 jasonwynn10 deleted the copy-plot branch Jan 10, 2020
@Aericio

This comment has been minimized.

Copy link
Collaborator

Aericio commented Jan 11, 2020

w-wwhat?!?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.