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-to-board to plot command #134

Open
Ana06 opened this issue Nov 3, 2017 · 12 comments
Open

Add --plot-to-board to plot command #134

Ana06 opened this issue Nov 3, 2017 · 12 comments
Labels
enhancement ✨ Enhancement good first issue 🥇 Really easy task, perfect for a first contribution help wanted 💕 PRs are welcome! Just comment in the issue saying you are working on it junior 👶 Easy task, perfect if you don't have much experience with the project

Comments

@Ana06
Copy link
Member

Ana06 commented Nov 3, 2017

This option would be also useful in this command. This will send the plotted burndown chart to the first card of the Done column.

@Ana06 Ana06 added enhancement ✨ Enhancement good first issue 🥇 Really easy task, perfect for a first contribution help wanted 💕 PRs are welcome! Just comment in the issue saying you are working on it junior 👶 Easy task, perfect if you don't have much experience with the project labels Nov 3, 2017
@ankitkataria
Copy link
Contributor

can i claim this issue too?

@Ana06
Copy link
Member Author

Ana06 commented Jan 29, 2018

@ankitkataria what about finishing #138 before starting with this one? 😉

@Ronaq13
Copy link
Contributor

Ronaq13 commented Feb 20, 2018

Can I take this issue

@Ana06
Copy link
Member Author

Ana06 commented Feb 20, 2018

@Ronaq13 sure! 👍

@Ronaq13
Copy link
Contributor

Ronaq13 commented Feb 20, 2018

Started working on it.

@Ronaq13
Copy link
Contributor

Ronaq13 commented Feb 23, 2018

@Ana06 I was trying my hands on it and have two doubts.

  1. When I tried to create a plot by trollolo plot 01 it gave me the following error:
    create_burndown.py", line 39 print args ^ SyntaxError: Missing parentheses in call to 'print'. Did you mean print(int args)?
    Rest of the integration I have done and I am able to run trollolo get-cards --board-id=MyTrelloWelcomeBoardID

When I fixed this by doing print(args), then it gave me the following error:

burndowndata.py", line 84, in getSprintData if day.has_key("fast_lane"): AttributeError: 'dict' object has no attribute 'has_key'

  1. Whenever I make changes do I need to re-build and re-install the gem everytime, to see changes?

@Ana06
Copy link
Member Author

Ana06 commented Feb 23, 2018

@Ronaq13 can you send a pull request with the changes? it is really difficult to find out what you exactly did without seeing the code 😉

@Ronaq13
Copy link
Contributor

Ronaq13 commented Feb 23, 2018 via email

@Ana06
Copy link
Member Author

Ana06 commented Feb 23, 2018

Sorry @Ronaq13 , I got you wrong. If you didn't change anything, and you find something that doesn't work, it may be a bug and then it would be better to open a separate issue 😉

@Ronaq13
Copy link
Contributor

Ronaq13 commented Feb 24, 2018

@Ana06 Ok it was my mistake, it was a python version conflict. Now I am back on the issue. I am facing a problem here, I am not able to figure out where I need to write the definition of plot-to-board option.

And also can you suggest anything for the following:
Whenever I make changes do I need to re-build and re-install the gem every time, to see changes?

Ronaq13 added a commit to Ronaq13/trollolo that referenced this issue Feb 26, 2018
This flag of plot command will send the plotted burndown chart to the first
card of the `Done` column.

close openSUSE#134
@Ana06
Copy link
Member Author

Ana06 commented Mar 1, 2018

@Ronaq13

Whenever I make changes do I need to re-build and re-install the gem every time, to see changes?

As I told you by email, yes. You can also write a test.

The rest, I'll answer in the PR

Ronaq13 added a commit to Ronaq13/trollolo that referenced this issue Mar 1, 2018
This flag of plot command will send the plotted burndown chart to the first
card of the `Done` column.

close openSUSE#134
Ronaq13 added a commit to Ronaq13/trollolo that referenced this issue Mar 1, 2018
This flag of plot command will send the plotted burndown chart to the first
card of the `Done` column.

close openSUSE#134
Ronaq13 added a commit to Ronaq13/trollolo that referenced this issue Mar 1, 2018
This flag of plot command will send the plotted burndown chart to the first
card of the `Done` column.

close openSUSE#134
@Ana06
Copy link
Member Author

Ana06 commented Oct 23, 2018

There is some work done in #184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ Enhancement good first issue 🥇 Really easy task, perfect for a first contribution help wanted 💕 PRs are welcome! Just comment in the issue saying you are working on it junior 👶 Easy task, perfect if you don't have much experience with the project
Projects
None yet
3 participants