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

Privateco Ⅰ (elekteblo pri videbligado de datenoj) #127

Merged
merged 9 commits into from
Jan 31, 2018

Conversation

interDist
Copy link
Collaborator

@interDist interDist commented Jan 10, 2018

La ideo estas doni al ĉiu uzanto pli grandan kontrolon kie la datenoj kiujn ŝli provizas al la sistemo de Pasporta Servo estas vidigataj. La agorda UI estas en formo de sekcio en la profilaj agordoj, kaj sekvas pli malpli la prototipon ĉe #78 (comment) enmetante la proponojn de @batisteo : divido inter ĉiuj uzantoj vs permesitaj uzantoj (nomitaj en la UI “gastoj kaj gastigantoj” – tio estas ŝanĝo traktenda tra tuta retejo en futuro), kaj tuja ebleco vidi kiu estas tiuj permesitaj uzantoj.

Jen ekzemploj de la agorda UI por la uzanto:
image
image

Tiu ĉi enmeto kaptas kelkajn birdojn: ĝi ebligas al Pasporta Servo bone respondi al postuloj de ĜDPR rilate publikigadon de personaj datenoj (#107 (comment)), ĝi donas al uzantoj la agrablan senton ke ili povas kontroli siajn datenojn, kaj ĝi faciligas la uzon de la retejo: ekzemple, se mi havas telefonnumeron kiun mi ne emas publikigi en la papera eldono aŭ loĝejon kiun mi portempe ne emas montri en la retejo, mi povas kaŝi tiujn datenojn anstataŭ forviŝi ilin kaj poste re-krei.

@interDist interDist requested a review from blfpd January 10, 2018 23:15
@pep8speaks
Copy link

pep8speaks commented Jan 10, 2018

Hello @interDist! Thanks for submitting the PR.

Line 261:9: E731 do not assign a lambda expression, use a def

Line 153:40: E203 whitespace before ':'
Line 369:5: E731 do not assign a lambda expression, use a def

Line 56:1: E305 expected 2 blank lines after class or function definition, found 1
Line 66:1: E305 expected 2 blank lines after class or function definition, found 1
Line 127:1: E305 expected 2 blank lines after class or function definition, found 1

Line 437:55: E203 whitespace before ':'


Edit: forigis line too long avertojn
-- @batisteo

@interDist
Copy link
Collaborator Author

@batisteo kial la stulta pep8speaks roboto ne honorigas la agordojn .pep8speaks.yml ?!

@blfpd
Copy link
Member

blfpd commented Jan 11, 2018

🎵 Iamo longe for, pep8speaks',
🎵 Iamo longe for, pep8speaks'! 🎶

Copy link
Member

@blfpd blfpd left a comment

Choose a reason for hiding this comment

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

Kelkaj aferoj kiuj ĝenerale ĝenas min:

  1. Plua uzo de jQuery, ĉar la jQuery-aj skriptoj estas tre malfacile por legi kaj plutrakti;
  2. Malklara intenco de la fandpeto;
  3. Pligrandigo de la kompleksa buĝeto;
  4. Ne sufiĉe da dokumentaj klasoj/funkcioj (kio estas venue(s) fakte?);
  5. Uzo de signaloj, ĉefe se absraktaj;
  6. Eble estas tempo por tranĉi hosting.views kaj hosting.forms;

Do, mi ne havas solvon pri tiuj rimarkoj, vi eĉ povas ignori ilin. Estas ĝenerala impreso nur legante la kodon.

while n:
r, n = r + n % 10, n // 10
n = r
return chr(ord('a') + n - 1) + chr(ord('Z') - n + 1)
Copy link
Member

Choose a reason for hiding this comment

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

Fakte, mi preferus ke la slug estu laŭ slugify(Profile.name) kaj slugify venas de awesome-slugify.

Almenaŭ la persona nomo ĉiam estu publika ĉe PS. Eĉ se malvera (KuboF, Stela… Sergei).

…(ref. a1e5c8f)

la etikedo estu uzata por konfidencaj datenoj kiuj montreblas nur en specifaj
kuntekstoj (laŭ la videbligo-agordoj). kiam uzanto ne ensalutis, la kunteksto
estas "tuta interreto" kaj en tiu kazo ni ne volas ke la dateno iam ajn aperu.
@interDist interDist merged commit 7c7a1c4 into tejoesperanto:master Jan 31, 2018
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