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

עקביות נתונים #450

Open
NoamGaash opened this issue Aug 10, 2022 · 0 comments
Open

עקביות נתונים #450

NoamGaash opened this issue Aug 10, 2022 · 0 comments

Comments

@NoamGaash
Copy link
Collaborator

NoamGaash commented Aug 10, 2022

בPR #403 היה נסיון לתקן התנהגות בעייתית -
יש hidden לנקודה גם בתוך מסלול וגם בתוך הנקודה עצמה (בpoints.json ובroutes.json).
זה אומר שכל עיר יכולה להיות כן\לא מוסתרת מספר פעמים, ובמיקומים שונים.
מסתבר שלא רק הhidden כפול - אלא גם המיקום (E,N), הצבע, וכו' - עד לרמת הקישור לwaze.
לא נשמע לי שיש בזה הגיון, לא מצאתי אף אחד שמסוגל להסביר את ההגיון, פרט לחריגים הבאים:

  1. יש הגיון מסויים בhidden לכל point בroute אם (ורק אם) יש מצבים שמטוס עובר מעל נקודה, אבל אנחנו לא רוצים שהוא יופיע ברשימת המטוסים שעוברים מעל אותה נקודה. לא נשמע לי שיש תרחיש כזה. אם כן - נדרש לוודא זאת מול המשתמש המבצעי. אם זו אכן דרישה - אז יש שתי אפשרויות:
    1. להחליט שהhidden של point יקבע אם הנקודה תופיע \ תעלם, ושהhidden של routes (של כל מטוס בנקודה) יקבע האם הנקודה תכיל את המטוס
    2. לדעתי מומלץ - לקבוע שאין צורך בhidden של points, ואם כל המטוסים שעוברים מעל העיר הם hidden מעל העיר בroutes אז העיר עצמה גם לא תופיע.
  2. בנוגע לצבעים -
    1. אפשרות ראשונה היא לשמור את הצבעים בpoints בלבד, ואז לא צריך צבע בroutes
    2. אפשרות שנייה היא שיהיו צבעים בroutes, ואז הגיוני שתהיה עיר שיש לה יותר מצבע אחד. במקרה כזה, כדאי להתייעץ עם UX - אני מאמין שעיר שיהיו לה הרבה צבעים תיראה ככה:
      image

בנוגע לשאר הנתונים הכפולים (E, N, wazeLink) הם צריכים להופיע רק בpoints, לא רואה סיבה לאף פתרון אחר.

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

No branches or pull requests

1 participant