List traffic bills instead of "Create Bill" if available #5616

Closed
BradHooper opened this Issue Jan 26, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@BradHooper

If a port has a Traffic Bill attached then it would be handy to link to the bill on the ports page instead of "Create Bill" link

http://i.imgur.com/Wf1Apbs.png

DO NOT DELETE THIS INFORMATION.

Please read this information carefully.

GitHub issues is for feature requests or bugs, please do not post issues asking for help or how to do X, Y or Z.
You can use our irc channel ##librenms on freenode to ask questions or our community site.

Please confirm each of the sections below by putting an x in the box like [x].

  • Is your install up to date? Updating your install
    Please do not submit an issue if your install is not up to date within the last 24 hours or on a stable monthly release.
  • Please include all of the information between the ==================================== section of ./validate.php which you can run from the cli.
  • Unless your issue is for a WebUI fix or feature then please provide ALL info asked for here.
  • Please provide as much detail as possible.
  • Please do NOT post more than 10 lines of debug information here, use a pastebin service or GitHub Gists.

@laf laf added the Feature label Jan 26, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Jan 28, 2017

Member

The issue I see with this is that you can assign a port to more than one bill so how do we decide which one to use?

Member

laf commented Jan 28, 2017

The issue I see with this is that you can assign a port to more than one bill so how do we decide which one to use?

@BradHooper

This comment has been minimized.

Show comment
Hide comment
@BradHooper

BradHooper Jan 28, 2017

If a single bill is available then maybe show the billing name, more than one bill then the billing numbers? Otherwise would the "ifcell" div be available to the right of the interface-desc TD be suitable?

If a single bill is available then maybe show the billing name, more than one bill then the billing numbers? Otherwise would the "ifcell" div be available to the right of the interface-desc TD be suitable?

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Jan 28, 2017

Member

PR inbound.

Member

laf commented Jan 28, 2017

PR inbound.

laf added a commit to laf/librenms that referenced this issue Jan 28, 2017

@f0o f0o closed this in #5653 Jan 29, 2017

f0o added a commit that referenced this issue Jan 29, 2017

@lock

This comment has been minimized.

Show comment
Hide comment
@lock

lock bot May 18, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

lock bot commented May 18, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

@librenms librenms locked as resolved and limited conversation to collaborators May 18, 2018

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