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

Review appstore development documentation changes

    Details

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

      Description

      There are known gaps in the current development documentation, specifically in Step 11: Local Email Setup. The email functionality is inconsistent it works in some environments and fails in others, sometimes without clear error messages.

      This intermittent behavior is not due to changes in Django itself. The send_mail() function used for email sending remains unchanged between Django 2.2 and 4.2, so the underlying framework should function consistently after the upgrade. The inconsistent behavior is likely related to incomplete or environment-specific setup steps that are currently missing or unclear in the documentation.

      To ensure reliability the development documentation must be updated and validated so that email functionality can be set up and tested reliably across different environments.

        Attachments

          Issue Links

            Activity

            Hide
            pkulzer Paige Kulzer (Inactive) added a comment -

            I'm very happy with the changes! Step 10 might keep evolving in the future since Google seems to be changing things constantly, but everything is up to date and easy to follow.

            Closing ticket!

            Show
            pkulzer Paige Kulzer (Inactive) added a comment - I'm very happy with the changes! Step 10 might keep evolving in the future since Google seems to be changing things constantly, but everything is up to date and easy to follow. Closing ticket!
            Hide
            sjagarap saideepthi jagarapu (Inactive) added a comment -

            attn Paige Kulzer Nowlan Freese

            Changes are done and final document is ready, review the document and let's see if we need anything else to be added.

            Show
            sjagarap saideepthi jagarapu (Inactive) added a comment - attn Paige Kulzer Nowlan Freese Changes are done and final document is ready, review the document and let's see if we need anything else to be added.
            Hide
            pkulzer Paige Kulzer (Inactive) added a comment -

            Some comments I had after running through this documentation with Dr. Freese:

            Step 10:
            #5: The details for setting up a project have been omitted here. Likely because it's straightforward, but since I'm prompted to enter a Project name, I'd like if one was provided in the documentation.
            #6-7: I think these can be skipped? When I make a project, I'm sent back to the developers console where there's a button at the top of the Credentials page that says "Create credentials". I can just click this and that is what #8 captures.
            #9: There is no radio button now. Instead, a warning has appeared that says, "To create an OAuth client ID, you must first configure your consent screen." And with that warning is a button labeled "Configure consent screen" which is what I need to click next. That should be #9. Then #10 should be the next item which is to configure the Google Auth Platform. There are fields here that should be outlined in the documentation, such as App Information, Audience (internal vs external), etc. After completing this project configuration, now a button is displayed that says "Create OAuth client". This is where the current #9 picks up. Again, when creating the OAuth client ID, I'd like if the Name field could be provided in the documentation (unless you'd like to use the default "Web Client 1").
            #12: This is a small point but we did have to search for Google+ API which was confusing to Dr. Freese at first. Might be good to note!

            The email set-up does appear to be working consistently now! My only request would be to include a screenshot of what your settings.ini might look like. You can use a fake email/password for documentation purposes, but it's nice to be able to check if I've got all the right lines in that file.

            Show
            pkulzer Paige Kulzer (Inactive) added a comment - Some comments I had after running through this documentation with Dr. Freese: Step 10: #5: The details for setting up a project have been omitted here. Likely because it's straightforward, but since I'm prompted to enter a Project name, I'd like if one was provided in the documentation. #6-7: I think these can be skipped? When I make a project, I'm sent back to the developers console where there's a button at the top of the Credentials page that says "Create credentials". I can just click this and that is what #8 captures. #9: There is no radio button now. Instead, a warning has appeared that says, "To create an OAuth client ID, you must first configure your consent screen." And with that warning is a button labeled "Configure consent screen" which is what I need to click next. That should be #9. Then #10 should be the next item which is to configure the Google Auth Platform. There are fields here that should be outlined in the documentation, such as App Information, Audience (internal vs external), etc. After completing this project configuration, now a button is displayed that says "Create OAuth client". This is where the current #9 picks up. Again, when creating the OAuth client ID, I'd like if the Name field could be provided in the documentation (unless you'd like to use the default "Web Client 1"). #12: This is a small point but we did have to search for Google+ API which was confusing to Dr. Freese at first. Might be good to note! The email set-up does appear to be working consistently now! My only request would be to include a screenshot of what your settings.ini might look like. You can use a fake email/password for documentation purposes, but it's nice to be able to check if I've got all the right lines in that file.
            Hide
            sjagarap saideepthi jagarapu (Inactive) added a comment -

            Updated the development guide - https://docs.google.com/document/d/1_9C03q6TD5wjLqfVLKsuDsIEQ4-qrS0JhONO5VBOwaA/edit?pli=1&tab=t.0

            Main issue causing inconsistency in sending mails locally -

            Email Port Differences: Local vs Production

            Local Dev: Only port 465 with SSL works. Port 587 hangs due to possible ISP, VPN, or firewall restrictions on outbound TLS connections.
            Production: Port 587 with TLS works as expected. Network/firewall policies differ from local environment.

            Show
            sjagarap saideepthi jagarapu (Inactive) added a comment - Updated the development guide - https://docs.google.com/document/d/1_9C03q6TD5wjLqfVLKsuDsIEQ4-qrS0JhONO5VBOwaA/edit?pli=1&tab=t.0 Main issue causing inconsistency in sending mails locally - Email Port Differences: Local vs Production Local Dev: Only port 465 with SSL works. Port 587 hangs due to possible ISP, VPN, or firewall restrictions on outbound TLS connections. Production: Port 587 with TLS works as expected. Network/firewall policies differ from local environment.

              People

              • Assignee:
                sjagarap saideepthi jagarapu (Inactive)
                Reporter:
                sjagarap saideepthi jagarapu (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: