Details
-
Type: Bug
-
Status: Closed (View Workflow)
-
Priority: Critical
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:
-
Story Points:1
-
Epic Link:
-
Sprint:Fall 5 : 14 Oct to 25 Oct, Fall 6 : 28 Oct to 8 Nov
Description
Steps to reproduce the issue:
1) Click on submit your app
2) Select an app to submit from "Choose File" option or Enter URL
3) Go back to the home page or click browser back button without submitting the app
This will create an entry in the pending apps table even if the app is not submitted for the release. Also, this will save the jar file for the app in pending_releases folder in s3 bucket.
Also, if the error page "Something is wrong" is displayed while submitting the app, the app entry will still be created in the AppPending table which can result in improper release of the app without user's approval.
Attachments
Issue Links
Activity
Riddhi Jagdish Patil (Inactive)
created issue -
Riddhi Jagdish Patil (Inactive)
made changes -
Field | Original Value | New Value |
---|---|---|
Epic Link | IGBF-1388 [ 17463 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Riddhi Jagdish Patil (Inactive)
made changes -
Summary | Returning from the page without submitting app creates entry in Pending app table | Returning from the submit app page without submitting app creates entry in Pending app table |
Riddhi Jagdish Patil (Inactive)
made changes -
Status | To-Do [ 10305 ] | In Progress [ 3 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Description |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser back button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. |
Riddhi Jagdish Patil (Inactive)
made changes -
Description |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser back button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser back button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. Also, if the error page "Something is wrong" is displayed while submitting the app, the app entry will still be created in the AppPending table which can result in improper release of the app without user's approval. |
Riddhi Jagdish Patil (Inactive)
made changes -
Labels | Advanced |
Riddhi Jagdish Patil (Inactive)
made changes -
Story Points | 1 | 2 |
Riddhi Jagdish Patil (Inactive)
made changes -
Description |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser back button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. Also, if the error page "Something is wrong" is displayed while submitting the app, the app entry will still be created in the AppPending table which can result in improper release of the app without user's approval. |
Steps to reproduce the issue:
1) Click on submit your app 2) Select an app to submit from "Choose File" option or Enter URL 3) Go back to the home page or click browser back button without submitting the app This will create an entry in the pending apps table even if the app is not submitted for the release. Also, this will save the jar file for the app in pending_releases folder in s3 bucket. Also, if the error page "Something is wrong" is displayed while submitting the app, the app entry will still be created in the AppPending table which can result in improper release of the app without user's approval. |
Riddhi Jagdish Patil (Inactive)
made changes -
Riddhi Jagdish Patil (Inactive)
made changes -
Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Story Points | 2 | 1 |
Ann Loraine
made changes -
Sprint | Fall 5 : 14 Oct to 25 Oct [ 76 ] | Fall 5 : 14 Oct to 25 Oct, Fall 6 : 28 Oct to 8 Nov [ 76, 77 ] |
Ann Loraine
made changes -
Rank | Ranked higher |
Ann Loraine
made changes -
Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
Ann Loraine
made changes -
Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
Ann Loraine
made changes -
Assignee | Ann Loraine [ aloraine ] |
Ann Loraine
made changes -
Assignee | Ann Loraine [ aloraine ] | Riddhi Jagdish Patil [ rpatil14 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Ann Loraine
made changes -
Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
Ann Loraine
made changes -
Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
Ann Loraine
made changes -
Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Ann Loraine
made changes -
Status | Post-merge Testing In Progress [ 10003 ] | Merged Needs Testing [ 10002 ] |
Pawan Bole (Inactive)
made changes -
Assignee | Pawan Bole [ pbole ] |
Pawan Bole (Inactive)
made changes -
Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Pawan Bole (Inactive)
made changes -
Status | Post-merge Testing In Progress [ 10003 ] | To-Do [ 10305 ] |
Pawan Bole (Inactive)
made changes -
Assignee | Pawan Bole [ pbole ] | Riddhi Jagdish Patil [ rpatil14 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Status | To-Do [ 10305 ] | In Progress [ 3 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Ann Loraine
made changes -
Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
Ann Loraine
made changes -
Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
Ann Loraine
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
Ann Loraine
made changes -
Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
Riddhi Jagdish Patil (Inactive)
made changes -
Assignee | Riddhi Jagdish Patil [ rpatil14 ] |
Ann Loraine
made changes -
Status | Reviewing Pull Request [ 10303 ] | Pull Request Submitted [ 10101 ] |
Ann Loraine
made changes -
Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
Ann Loraine
made changes -
Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
Pawan Bole (Inactive)
made changes -
Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Pawan Bole (Inactive)
made changes -
Assignee | Pawan Bole [ pbole ] |
Pawan Bole (Inactive)
made changes -
Resolution | Done [ 10000 ] | |
Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
Pawan Bole (Inactive)
made changes -
Assignee | Pawan Bole [ pbole ] | Riddhi Jagdish Patil [ rpatil14 ] |
Suggestion for how to address this problem:
Add a new boolean value field to AppPending class in submit_ap/models.py:
How this would work:
When user clicks "Next" on the "submit app" page UI, then App Store creates an entry in the AppPending table for the App. This is already being done. What's new is that the new row of data will include "submitter_approved" field set to False, the default.
Next, App Store displays the pre-submission feedback page, which shows information about this "pending" App.
If the user clicks "Submit", then "submitter_approved" changes to "True", along with other actions already being performed, such as sending an email to the user and to our app approvers ("admin" users).
If the user clicks "Cancel", then the row is deleted from the database and the uploaded jar file gets removed from the system.
If the user does nothing, then the field "submitter_approved" remains False.
Separately, we implement a data clearing mechanism that removes all stale AppPending rows (and jar files) with false "submitter_approved" and "created" dates older than a pre-set amount, e.g., 24 hours.
If a user gets distracted or disrupted during this process, they might repeat the app submission process from the beginning (most likely) or might return directly to this feedback page if it is still open.
If they repeat the process from the beginning before we get around to clearing out the old, stale data, then the previously uploaded jar file and the previously created entry in the pending apps database table will simply be over-written.
If they come back to the "feedback" page and click "Submit" after we have cleared the data, then we display a page that says something like "Sorry, this App is not longer in our system because too much time has passed since you first uploaded it. No problem! Please go back and try again."
Also, our admin users who can approve apps should only see Apps that have been approved by their submitter - i.e., submitter_approved is True. All other Apps could be shown in a different category on that page, or we could not show them at all. If somebody has changed their mind about submitting and an App or is maybe not sure they want to submit it, we should probably ignore it.
I think we should do something like the above rather than add more complexity to the system by using cookies and introducing state to user's browsers. Also, if we start using cookies, we have to show a disclaimer, which I feel are a bit annoying and distracting.