-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[$500] Compose-Profile in header is shown when tapping on deep link #32905
Comments
Triggered auto assignment to @sonialiap ( |
Job added to Upwork: https://www.upwork.com/jobs/~01e3a5ca00ba4560c8 |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha ( |
Why shouldn't the Profile be shown here?) But if we need to change the title if the profile is not valid, then ... ProposalPlease re-state the problem that we are trying to solve in this issue.Compose-Profile in header is shown when tapping on deep link What is the root cause of that problem?We show profile text by default What changes do you think we should make in order to solve the problem?Since instead of the NoteFound wrapper we use a separate component we can update the title and add the same condition using Line 131 in 2165e74
For example (instead '' we can use another text)
Line 152 in 2165e74
What alternative solutions did you explore? (Optional)NA |
ProposalPlease re-state the problem that we are trying to solve in this issue.Compose-Profile in header is shown when tapping on deep link What is the root cause of that problem?The root problem of the issue is that the link used in the example is used for opening a user profile in the Right modal: Lines 28 to 31 in 83bd22f
Since the userId is not found we are shown the error screen. What changes do you think we should make in order to solve the problem?We could validate userIds before navigation. We have the helper function App/src/libs/ValidationUtils.ts Lines 360 to 362 in 83bd22f
What alternative solutions did you explore? (Optional) |
📣 @apps-ferreiro! 📣
|
ProposalPlease re-state the problem that we are trying to solve in this issue.When user taps on this deep link, "profile" is shown as title in header. What is the root cause of that problem?We're using inconsistent pattern in What changes do you think we should make in order to solve the problem?Use This also makes the RHN not found behavior consistent everywhere since in all other LHN not found page, it has the subtitle, only the Profile page "NotFound" state does not have subtitle. What alternative solutions did you explore? (Optional)If instead we want the Profile page "NotFound" state not to have subtitle, we can modify |
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
ProposalPlease re-state the problem that we are trying to solve in this issue.Compose-Profile in header is shown when tapping on deep link What is the root cause of that problem?Currently when the profile page loads we validate only if the account ID in the URL is a number, and if so we add the default data to Onyx (this also causes another issue, showing the profile as hidden when the profile does not exist). Also, the bug reported only happens when the account ID has text or the ID is very large. See the video: Staging.test.mp4What changes do you think we should make in order to solve the problem?We should look into props.personalDetails in ProfilePage.js and verify if AccountID is in the list, with this we can determinate if the user exists in our DB, something like:
Then we can determine better if we should openPublicProfilePage and if not we'll need to set the isLoading variable in Onyx as false like:
Finally, we should show the warning in this way:
If shouldShowProfileError is true, we should hide the other components like:
See the final result: Fixed.bug.mp4 |
@getusha what do you think of the above proposals? |
Reviewing |
Raised a discussion https://expensify.slack.com/archives/C01GTK53T8Q/p1702639941065949, to determine if this is worth fixing. |
This doesn't fit into any of the waves. I agree with the votes to :donothing:. Closing out |
@sonialiap, just curious, if the issue is closed out as completed and "donothing" is the course of action taken, is there a bounty awarded to any of the proposals? |
📣 @cartersteinhoff! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
@sonialiap I am interested in contributing proposals to issues that are raised for this app, but when I see instances such as these it is discouraging because as contributors we can do everything right on our end and propose good solutions, but as it turns out there is no real potential for payout. This kind of situation decreases the expected value of answering these issues with proposals. Hopefully, these kinds of situations are kept to minimum with plans to reduce their occurrences in the future? |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.11
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:
https://staging.new.expensify.com/a/hello
Expected Result:
When user taps on this deep link, "profile" must not be shown as title in header.
Actual Result:
When user taps on this deep link, "profile" is shown as title in header.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6310018_1702360305937.pro_ile_header.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: