Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2922

Explain and/or revise architecture diagram (fig 1)

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      I think maybe Figure 1 - the architecture diagram - needs revision.

      The client / server dividing line is a bit confusing. IGB is show on the server side of the dividing line, which does not make sense to me because IGB is not a client-side program. Similarly, "template" is on the client side of the dividing line, but Django templates are entirely server-side. That's the entire point of them, I thought - they are never exposed to the Web browser.

      The meaning of the two boxes on the server side is also confusing. Are those boxes meant to make the figure look nicer, or do they have a meaning with respect to the architecture?

        Attachments

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-1551 [ 17629 ]
          ann.loraine Ann Loraine made changes -
          Assignee Karthik Raveendran [ karthik ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          ann.loraine Ann Loraine made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          ann.loraine Ann Loraine made changes -
          Description I think maybe Figure 1 - the architecture diagram - needs revision. At the very least, there needs to be a narrative explaining what it means.

          The client / server dividing line is very very confusing. IGB is show on the server side of the dividing line, which does not make sense to me because IGB is not a client-side program. Similarly, "template" is on the client side of the dividing line, but Django templates are entirely server-side. That's the entire point of them, I thought - they are never exposed to the Web browser.

          The meaning of the two boxes on the server side is also confusing. Are those boxes meant to make the figure look nicer, or do they have a meaning with respect to the architecture?

          Please provide a detailed legend for the figure, explaining what each part represents. If you try to put into words what the pictures mean, you will likely be able to think of ways to make the image more clear.
          I think maybe Figure 1 - the architecture diagram - needs revision.

          The client / server dividing line is a bit confusing. IGB is show on the server side of the dividing line, which does not make sense to me because IGB is not a client-side program. Similarly, "template" is on the client side of the dividing line, but Django templates are entirely server-side. That's the entire point of them, I thought - they are never exposed to the Web browser.

          The meaning of the two boxes on the server side is also confusing. Are those boxes meant to make the figure look nicer, or do they have a meaning with respect to the architecture?
          ann.loraine Ann Loraine made changes -
          Attachment Figure1.png [ 16780 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Made a new version, which I hope is better. It is attached.

          Show
          ann.loraine Ann Loraine added a comment - Made a new version, which I hope is better. It is attached.
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          ann.loraine Ann Loraine made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ann.loraine Ann Loraine made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Previous version

          Show
          ann.loraine Ann Loraine added a comment - Previous version
          ann.loraine Ann Loraine made changes -
          Attachment 1.jpg [ 16781 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Note: If reviewers ask, we can further improve the figure for clarity.

          Show
          ann.loraine Ann Loraine added a comment - Note: If reviewers ask, we can further improve the figure for clarity.

            People

            • Assignee:
              ann.loraine Ann Loraine
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: