Skip to content
This repository was archived by the owner on May 18, 2022. It is now read-only.

Trello#1

Open
arpitrathi wants to merge 5 commits intoworkattech:masterfrom
arpitrathi:master
Open

Trello#1
arpitrathi wants to merge 5 commits intoworkattech:masterfrom
arpitrathi:master

Conversation

@arpitrathi
Copy link
Copy Markdown

No description provided.

from Trello.Models.Library import IdGenerator


class Cards(object):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Card

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do not use plural

from Trello.Models.Library import IdGenerator


class Lists(object):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

List

@gcnit
Copy link
Copy Markdown
Collaborator

gcnit commented Oct 23, 2019

Hi @arpitrathi sorry for the delay in code review.

The code looks good. Gave a few comments.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants