Skip to content

Conversation

@kaizencodes
Copy link
Contributor

for 3185

Copy link
Member

@Azdaroth Azdaroth left a comment

Choose a reason for hiding this comment

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

💪

Copy link
Member

@Azdaroth Azdaroth left a comment

Choose a reason for hiding this comment

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

Actually we are missing changelog here :P

@kaizencodes
Copy link
Contributor Author

@Azdaroth done


## 2017-10-02
* [api improvement] Add `add_bookings_fees` endpoint
* [api improvement] Add `remove_bookings_fees` endpoint
Copy link
Member

Choose a reason for hiding this comment

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

Why duplicating? :P

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Azdaroth Seemed like the convention, I can merge it if you'd like

Copy link
Member

Choose a reason for hiding this comment

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

Wow, is it really duplicated with the same case for 2 different labels?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Azdaroth done

Copy link
Member

@Azdaroth Azdaroth left a comment

Choose a reason for hiding this comment

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

💪

@Azdaroth
Copy link
Member

Azdaroth commented Oct 6, 2017

@kaizencodes Can you please merge and release it?

@kaizencodes kaizencodes merged commit 14d2814 into master Oct 6, 2017
@kaizencodes kaizencodes deleted the add-bookings-fees branch October 6, 2017 15:53
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