Skip to content

Print labels with username and current date.#43

Open
YtvwlD wants to merge 2 commits intomainfrom
issue39
Open

Print labels with username and current date.#43
YtvwlD wants to merge 2 commits intomainfrom
issue39

Conversation

@YtvwlD
Copy link
Member

@YtvwlD YtvwlD commented Sep 24, 2017

This implements issue #39. But perhaps this is too simple.
Do we need to configure the labello API endpoint?
Do users need to select whether something is public or private?

@YtvwlD YtvwlD force-pushed the issue39 branch 2 times, most recently from f439dde to 20313ab Compare September 28, 2017 10:12
@YtvwlD
Copy link
Member Author

YtvwlD commented Sep 28, 2017

BLOCKED This needs to be rebased after PR #45 is merged.
BLOCKED This needs #47.

def print_label
@user = User.find(params[:id])
text = "#{@user.name} #{Date.today.iso8601}"
uri = URI('http://labello/print')
Copy link
Member

Choose a reason for hiding this comment

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

configureable!

Copy link
Member Author

Choose a reason for hiding this comment

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

So, we need a configuration file. But we need this for #5 anyway.

@YtvwlD YtvwlD force-pushed the issue39 branch 2 times, most recently from 77e3834 to d227b7d Compare September 29, 2017 19:09
@YtvwlD YtvwlD force-pushed the issue39 branch 2 times, most recently from e37d1a5 to 68749b7 Compare October 1, 2017 10:01
@YtvwlD YtvwlD added the blocked label Oct 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants