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

Fehlermeldung bei Einstellungen / Kontakte suchen #64

Closed
Tracked by #61
xwolfde opened this issue Oct 22, 2024 · 13 comments
Closed
Tracked by #61

Fehlermeldung bei Einstellungen / Kontakte suchen #64

xwolfde opened this issue Oct 22, 2024 · 13 comments
Assignees
Labels
bug Something isn't working

Comments

@xwolfde
Copy link
Member

xwolfde commented Oct 22, 2024

Unter der Eingabe erscheint beim erstaufruf "An error occured during the request":
grafik

@xwolfde xwolfde mentioned this issue Oct 22, 2024
23 tasks
@xwolfde xwolfde added the bug Something isn't working label Oct 22, 2024
@xwolfde
Copy link
Member Author

xwolfde commented Oct 22, 2024

Bei der Suche mit Vorname + Nachname gibt es eine Fehlermeldung:

"Error: Error decodong JSON data"
grafik

Ausserdem erscheint oben ein "Die Einstellungen wurden gespeichert."

@xwolfde
Copy link
Member Author

xwolfde commented Oct 22, 2024

Wenn ich nach EIngabe von Vornamen und Nachnamen statt auf Search "klicke", auf "RETURN" klicke, erscheint "An error occurred during the request." und oben "Die Einstellungen wurden gespeichert."
grafik

@xwolfde
Copy link
Member Author

xwolfde commented Oct 22, 2024

Bei einer Suche mit ID oder E-Mail erscheint ein eindeutiger Treffer. Und was fange ich damit dann an? Da gibtg es dann keine weitere Aktion, wie ich den gefundenenn EIntrag dann übernehme? Wäre schön wenn es ein Button gäbe der dann zum Anlegen führt:
grafik

Desweiteren ist die Id keine IdM-Kennung. Das ist falsch bezeichnet.

Ausserdem auch hier die Meldung: "Die Einstellungen wurden gespeichert." - Diese meldung kommt, wenn die Options gespeichert werden.

@xwolfde
Copy link
Member Author

xwolfde commented Oct 22, 2024

Wenn ich Givenname + Familiy Name + E-Mail eingebe mit einem Mix (Wiese als Givenname + Wolfgang als Family Name + [email protected]), der keinen treffer liefern kann, erhalte ich aus Fehlerausgabe: Error: Error decoding JSON data.

Wenn ich es richtig rum eingeben, dann ebenfalls: Error: Error decoding JSON data.

Wenn ich nur die Mailadresse ovn mir eingebe und auf "search" klicke, kriege ich meine Result.

Wenn ich da aber auf "RETURN" in der Tastatur klick nach der Mailadresse, kriege ich "An error occurred during the request."

Wenn ich in der allgemeinen ersten EIngabebox meinen Vorname und Nachnamen eingebe, kriege ich meinen Treffer. Darf aber nur auf "Search" klicken, Tastatur-Return führt zu Fehler.

@FitoreGashi
Copy link

Ich werde die Standardfunktion der "Enter"-Taste, die mit dem Speichern der Einstellungen im ersten Tab verknüpft ist, korrigieren und die Übersetzung der Fehlermeldung überprüfen. Außerdem werde ich prüfen, warum die Suche nach weiteren Feldern nicht genauer spezifiziert wird. Der Button zum Hinzufügen eines Kontakts wurde im Issue nicht erwähnt, aber wir können diesen ebenfalls hinzufügen.

@xwolfde
Copy link
Member Author

xwolfde commented Oct 23, 2024

Stand im dazugehörigen Issue #26 , letzter Satz.

FitoreGashi added a commit that referenced this issue Oct 24, 2024
@MManthey
Copy link

@xwolfde wie soll sich das Plugin verhalten, wenn versucht wird, eine Person zu importieren, die bereits existiert? Ignorieren/Aktualisieren/Einfach neu anlegen(Duplikat)? Und reicht ein button pro Person zum Importieren oder soll es auch eine Funktion geben, die alle gefundenen Personen importiert?

@xwolfde
Copy link
Member Author

xwolfde commented Oct 24, 2024

Am besten bei bestehenden Personen einfach kein Import anbieten, sondern da ein Texthinweis machen "Perosn existiert bereits". Schön wäre eine Verlinkung dann gleich zum bestehenden Eintrag (Editor).

MManthey added a commit that referenced this issue Nov 5, 2024
@MManthey MManthey added the Abnahmetest Dieser Issue wurde vom Entwickler als fertig gesehen und kann getestet/abgenommen werden label Nov 6, 2024
@xwolfde
Copy link
Member Author

xwolfde commented Nov 8, 2024

Also in der aktuellen Version ist es seltsam:

Wenn ich die "Kontakte suchen" Lasche aufrufe, sehe ich ja eine Liste an Leuten. vgl #84) .

  • Gebe ich nun meine Mailadresse in der Suche ein (und ich existiere bereits als Kontakt): findet er mich nicht, und in der Console erscheint die Meldung:
grafik

gebe ich eine andere person ein, von der ich weiß, dass sie existiert und öfentlich ist (anhand der Liste die ich zuvor bekam):
mit dem Namen "Marcel Ritter":
grafik

Oder der Mailadresse auch nicht:
grafik

@xwolfde xwolfde removed the Abnahmetest Dieser Issue wurde vom Entwickler als fertig gesehen und kann getestet/abgenommen werden label Nov 8, 2024
@xwolfde xwolfde assigned MManthey and unassigned xwolfde Nov 8, 2024
@MManthey
Copy link

MManthey commented Nov 12, 2024

Also in der aktuellen Version ist es seltsam:

Wenn ich die "Kontakte suchen" Lasche aufrufe, sehe ich ja eine Liste an Leuten. vgl #84) .

  • Gebe ich nun meine Mailadresse in der Suche ein (und ich existiere bereits als Kontakt): findet er mich nicht, und in der Console erscheint die Meldung:
grafik

Aus dieser Eingabe ergibt sich folgende API Anfrage:

curl -X 'GET' \
  'https://api.fau.de/pub/v1/opendir/persons?lq=email%3Dwolfgang.wiese%40fau.de' \
  -H 'accept: application/json' \
  -H 'X-API-KEY: '

Die E-Mail ist bei dieser Person nicht hinterlegt, daher das Ergebnis.

Hier sind die Daten, welche die API zu dir liefert:

{
  "data": [
    {
      "@context": {
        "@base": "https://api.fau.de/pub/v1/opendir/"
      },
      "@type": "Person",
      "@id": "persons/14216341f6",
      "identifier": "14216341f6",
      "givenName": "Wolfgang",
      "familyName": "Wiese",
      "contacts": [
        {
          "@type": "Contact",
          "@id": "contacts/a288d74c81",
          "identifier": "a288d74c81",
          "organization": {
            "@type": "Organization",
            "@id": "organizations/f54753b711",
            "identifier": "f54753b711",
            "longDescription": {
              "de": "Ausbildung & Information",
              "en": "Ausbildung & Information"
            },
            "name": "Ausbildung & Information"
          },
          "function": "leader",
          "functionLabel": {
            "de": "Leitung",
            "en": "Head"
          }
        }
      ],
      "personalTitleSuffix": "Akademischer Oberrat"
    }
  ],
  "pagination": {
    "limit": 20,
    "count": 1,
    "total": 1,
    "offset": 0,
    "remaining": 0,
    "page": 1
  }
}

gebe ich eine andere person ein, von der ich weiß, dass sie existiert und öfentlich ist (anhand der Liste die ich zuvor bekam): mit dem Namen "Marcel Ritter": grafik

Der Console entnehme ich, dass hier der Name in das ID Feld eingegeben wurde. Eine Person mit der ID 'Marcel Ritter' existiert nicht.

Oder der Mailadresse auch nicht: grafik

Auch hier wieder - bei Marcel Ritter sind keine E-Mail Informationen hinterlegt. Hier der Eintrag, den die API liefert zu der Person:

{
  "data": [
    {
      "@context": {
        "@base": "https://api.fau.de/pub/v1/opendir/"
      },
      "@type": "Person",
      "@id": "persons/b51a219844",
      "identifier": "b51a219844",
      "givenName": "Marcel",
      "familyName": "Ritter",
      "contacts": [
        {
          "@type": "Contact",
          "@id": "contacts/c99925aaaf",
          "identifier": "c99925aaaf",
          "organization": {
            "@type": "Organization",
            "@id": "organizations/a51724f62c",
            "identifier": "a51724f62c",
            "longDescription": {
              "de": "Regionales Rechenzentrum Erlangen (RRZE)",
              "en": "Regionales Rechenzentrum Erlangen (RRZE)"
            },
            "name": "Regionales Rechenzentrum Erlangen (RRZE)"
          },
          "function": "leader",
          "functionLabel": {
            "de": "Leitung",
            "en": "Head"
          }
        }
      ]
    }
  ],
  "pagination": {
    "limit": 20,
    "count": 1,
    "total": 1,
    "offset": 0,
    "remaining": 0,
    "page": 1
  }
}

Bitte klaere uns auf, falls hier etwas falsch verstanden wurde.

@xwolfde
Copy link
Member Author

xwolfde commented Nov 12, 2024

Ich glaub ich hatte es selbst nicht richtig verstanden, aber langsam blick ich da durch. Vgl auch #97 .
Auch ist das Beispiel in der Doc der API leider etwas irreführend, weil da eine Mailadresse stand.

Es ist so, dass die Mailadresse, die man zur Veröffentlichung angeben kann (schema person) nicht an der Person hängen muss, sondern an der Funktion in einer Organisation zu der man gehört. Somit kann es sein, dass die Mailadresse nicht in den schema.person daten drin steckt, sondern unter denn Schema Contact bzw darin unter Workplaces.

Das ist ein wenig doof. Da werde ich mal die Kollege nachfragen, ob man das auch ändern kann.
Eine Suche nach Mail sollten wir schon haben, weil das so ziemlich das einzig sinnvolle ist, was keine Tippfehlerprobleme aufweist.

@xwolfde
Copy link
Member Author

xwolfde commented Nov 15, 2024

Hab jetzt Klarheit:

Die Abfrage auf den Persons -Endpoint, wie bspw.
https://api.fau.de/pub/v1/opendir/persons?lq=email=wolfgang.wiese%40fau.de
liefert keine Rückmeldung, da meine Mailadresse dem Arbeitsplatz zugeordnet ist.
Daher fehlt diese dann dort.
Die Mail-Suche beim Endpoint /persons findet also nur die wenigen Personen, bei denen keine Arbeitsplatzzuordnung vorhanden ist!

Um einen Personen-Identifier anhand der E-Mail-Adresse zu finden müsste man also 2 Suchen machen:

Zuerst im Contacts Endpoint suchen:
https://api.fau.de/pub/v1/opendir/contacts?lq=workplaces.mails%3Dwolfgang.wiese%40fau.de

Und erst danach, wenn man nichts fand, im Persons Endpoint:
https://api.fau.de/pub/v1/opendir/persons?lq=email%3Dwolfgang.wiese%40fau.de

@xwolfde
Copy link
Member Author

xwolfde commented Dec 6, 2024

Suche funktioniert jetzt...

Allerdings eine kleine Unschönheit: Wenn man nach Vornamen und Nachnamen sucht, ist es ein Exact-Case.
Sprich: Wenn ich nach Vorname "Wolf" und Nachname "Wiese" suche, finde ich mich nicht.
Ich muss beim Vornamen den exakten und ganzen Vornamen eingeben.
Ebenso ist die Suche Case Sensitiv.

Das war so nicht gefordert, wäre aber dennoch schön. Ich setze diesen Issue daher als erledigt an, mache aber ein Enhancement-Issue um die Suchabfrage bei den Feldern Vornamen, Nachname und Mail Case Insensitive zu machen und dass die Vornamen und Nachnamen eine Teilsuche ermöglichen, wenn mindestens 2 Zeichen eingegeben wurden. (Kürzester Name könnte ja zweibuchstabig sein): #117

@xwolfde xwolfde closed this as completed Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants