Skip to content
This repository has been archived by the owner on Dec 2, 2021. It is now read-only.

use the URL shortner #81

Closed
rahgurung opened this issue Sep 2, 2020 · 5 comments
Closed

use the URL shortner #81

rahgurung opened this issue Sep 2, 2020 · 5 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@rahgurung
Copy link
Member

Make sure graph 180 works.
@rahgurung rahgurung added enhancement New feature or request help wanted Extra attention is needed labels Sep 2, 2020
@rahgurung
Copy link
Member Author

@baljinderxd would you like to give it a try?

@baljinderxd
Copy link
Contributor

@baljinderxd would you like to give it a try?

Yes sure. Will try and make a PR by end of the day tomorrow.😬

@rahgurung
Copy link
Member Author

rahgurung commented Sep 2, 2020 •
edited

Yes sure. Will try and make a PR by end of the day tomorrow.

Great, we will need to add this for all graph commands later but for now graph command is the only priority.

@rahgurung
Copy link
Member Author

Till then I'm restricting the graph N command to a smaller value of N to prevent crashes.

@baljinderxd
Copy link
Contributor

graph command is the only priority.

Okay then, I will implement the shortener for graph command only in the PR.

@baljinderxd baljinderxd mentioned this issue Sep 3, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rahgurung @baljinderxd