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

Rename empty choice option from Not Disclosed to Not Available #30

Open
paravoid opened this issue Oct 23, 2018 · 9 comments
Open

Rename empty choice option from Not Disclosed to Not Available #30

paravoid opened this issue Oct 23, 2018 · 9 comments
Labels
Minor Up to 4 hours Requires Notice Implementing this ticket requires user be notified

Comments

@paravoid
Copy link
Contributor

OK, I expect this to a) be controversial, b) to have to wait for the "new translation server" (like in #24), but here goes anyway:

For a lot of the choice fields, the choices are:

    ('', _('Not Disclosed')),
    ('Not Disclosed', _('Not Disclosed')),

This is confusing, and results in e.g. search forms that look like this:
django-peeringdb

I'd argue that it would be better to rename the former to "Not Available", as that's what it really is, right? Thoughts?

[ Also, while we're here... why the Camel Case capitalization on this and other strings (such as "Not Required", "Heavy Outbound" etc.)? ]

@grizz
Copy link
Member

grizz commented Oct 26, 2018

I think they're both the same, we should just normalize down to "" for it -- that's leftover cruft from PDBv1

@peeringdb/pc thoughts?

@mcmanuss8
Copy link

+1 normalize to empty string

@mcmanuss8
Copy link

(RE: Camel Casing, I would fork that to a new issue @paravoid - that's more controversial ;)

@grizz
Copy link
Member

grizz commented Oct 26, 2018

I missed the camel casing thing on the end (and it's not camel casing per se, it's just capitalized words) -- created #32

@arnoldnipper
Copy link

+1 normalize to empty string

@paravoid
Copy link
Contributor Author

paravoid commented May 1, 2020

Ping!

@arnoldnipper
Copy link

Shouldn't that be in peeringdb instead of django-peeringdb?

@paravoid
Copy link
Contributor Author

paravoid commented May 1, 2020

The ambiguous constant mapping is in this project, under const.py:

RATIOS = (
    ("", _("Not Disclosed")),
    ("Not Disclosed", _("Not Disclosed")),
[…]

If that needs to be normalized so then… yeah, probably cross-project effort.

@grizz
Copy link
Member

grizz commented Aug 25, 2020

@arnoldnipper as @paravoid said, but the peeringdb project uses this code, so it's only here.

@grizz grizz added Minor Up to 4 hours Requires Notice Implementing this ticket requires user be notified labels Aug 25, 2020
@grizz grizz added this to the 4 Ready for Implementation milestone Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Up to 4 hours Requires Notice Implementing this ticket requires user be notified
Projects
None yet
Development

No branches or pull requests

4 participants