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

Bitly docs #22

Merged
merged 3 commits into from
Jul 8, 2020
Merged

Bitly docs #22

merged 3 commits into from
Jul 8, 2020

Conversation

Shraddha2104
Copy link
Contributor

Hi @tanay1337 ,
✨ I have added basic instructions for Bitly
Let me know if any changes are needed!

Copy link
Contributor

@smamudhan smamudhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @Shraddha2104, thank you for contributing!
Could you make the following changes before we merge into master?


## Example Usage

This workflow shows you how to create a new link. You can also find the [workflow](https://n8n.io/workflows/442) on this website. This example usage workflow uses the following two nodes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you change it to "on the website"?


This workflow shows you how to create a new link. You can also find the [workflow](https://n8n.io/workflows/442) on this website. This example usage workflow uses the following two nodes.

- [Start](../../core-nodes/Start)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you make sure it's similar to other nodes' documentation?

[Bitly](https://bitly.com/) is URL shortening service and a link management platform that allows users to shorten, create and share trusted, powerful links for businesses.

::: tip 🔑 Credentials
The instructions for authenticating with Bitly are [here](../../../credentials/Bitly).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you make sure it's similar to other nodes' documentation?


### 2. Bitly node

1. First, ensure that you have entered authentication credentials through [this tutorial](../../../credentials/Bitly/README.md).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you make sure it's similar to other nodes' documentation?

Comment on lines 39 to 40
2. To create a link, select the *Link* option in the *Resource* field.
3. Select the *Create* option in the *Operation* field.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are default options for this node, could you remove them?

1. First, ensure that you have entered authentication credentials through [this tutorial](../../../credentials/Bitly/README.md).
2. To create a link, select the *Link* option in the *Resource* field.
3. Select the *Create* option in the *Operation* field.
4. Enter the long URL in the *Long URL* field.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you change this to: Enter the URL in the *Long URL* field.?

@Shraddha2104
Copy link
Contributor Author

Hello @Shraddha2104, thank you for contributing!
Could you make the following changes before we merge into master?

Hey @smamudhan
I have updated the docs accordingly. Let me know if any changes are needed!

@tanay1337 tanay1337 merged commit 8a4ae06 into n8n-io:master Jul 8, 2020
@tanay1337
Copy link
Contributor

Merged, thanks a lot @Shraddha2104 🙌

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.

3 participants