-
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
[$50] "Your data is protected" link is broken #48031
Comments
Triggered auto assignment to @adelekennedy ( |
Job added to Upwork: https://www.upwork.com/jobs/~0194501906e3889a0d |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh ( |
Upwork job price has been updated to $50 |
Edited by proposal-police: This proposal was edited at 2024-08-27 09:59:28 UTC. Proposal Please re-state the problem that we are trying to solve in this issue. What is the root cause of that problem? The error persists even when the link is manually pasted into a new browser tab, indicating a problem outside of the our Expensify application. The root cause of "Error 1014" is likely related to a CNAME Cross-User Banned issue on Cloudflare. This error typically occurs when one domain points to another via a CNAME record, and the domains are managed under different Cloudflare accounts. The issue seems to originate from the configuration of the community.expensify.com domain or its Cloudflare settings. What changes do you think we should make in order to solve the problem? If we want to use the same link then contact the administrators responsible for community.expensify.com. Inform them of the "Error 1014" and request that they review the CNAME configurations within their Cloudflare account. What alternative solutions did you explore? App/src/pages/ReimbursementAccount/BankAccountStep.tsx Lines 168 to 173 in d5cfecf
Consider providing an alternative link for users to access the content. Replace above link with https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security |
Expensify/App Your proposal will be dismissed because you did not follow the proposal template. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Link is broken when clicking What is the root cause of that problem?I think we already migrated from Because when we open the What changes do you think we should make in order to solve the problem?We need to ask Expensify internal team for the right article inside the And update the below code: App/src/pages/ReimbursementAccount/BankAccountStep.tsx Lines 168 to 172 in d5cfecf
What alternative solutions did you explore? (Optional) |
@adelekennedy, it seems that the URL is outdated and needs to be updated. Can you confirm if this is the correct link? |
@ijmalik proposal looks good to me. We just need to confirm the correct link #48031 (comment) 🎀 👀 🎀 C+ reviewed |
Triggered auto assignment to @blimpich, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@fedirjh Ijmalik updated the alternative solution very similar to my solution, can you please look at the timestamps Before he updated the proposal, here's his original alternative solution:
|
@NJ-2020 I have already checked the timestamps, and the alternative solution looked correct to me as it's a straightforward fix. |
@fedirjh But as you can see the alternative solution it's almost exact same, the only difference he provide the link: |
@NJ-2020 Following The contributing guidelines :
I didn’t really notice any considerable difference between the two solutions. The solution is simple: we just need to update the link. |
@fedirjh Yes I've read, but he didn't mention about that earlier |
Hmmm, he mentioned in his initial proposal that we can use an alternative link. Then he provided the link for us. In any case, @blimpich will make the final decision. |
Ok, Thanks |
@adelekennedy the community site is being/has been deprecated. For the OP and the expected results, we should include and update in the triage the correct link we expect it to have been replaced by. If you're unsure what that replacement link is, #expensifyhelp would be a good place to ask. 👍 |
@adelekennedy do you know if this would be the correct link to replace this with? https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security |
📣 @ijmalik You have been assigned to this job! |
@ijmalik, @blimpich, @fedirjh, @adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@ijmalik are you able to raise a PR soon? |
< @ijmalik are you able to raise a PR soon? |
Hi @blimpich , I have applied for the Upwork job, but it seems my status hasn’t been updated here yet. Please let me know if any further action is required from my side. |
I don't think you need to do anything else. @adelekennedy are you able to double check that @ijmalik has done all the steps they need to do in order to get paid via Upwork? I believe they have. |
sorry! This totally slipped off my radar - I've hired you in Upwork @ijmalik |
This was also deployed to production, so @adelekennedy we should be ready to pay out I believe. |
I'm sorry about this @ijmalik, it looks like some of our automations failed so the payment process for this didn't work as smoothly as it should've. We will make sure to pay you and the C+ who reviewed your work. |
Thank you very much for the update. I completely understand, and there's no issue with the delay. I appreciate your efforts in resolving this. |
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: 9.0.24-2
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: @hannojg
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724679642375049
Action Performed:
Your Data Is Secure
linkExpected Result:
https://community.expensify.com/discussion/5677/deep-dive-how-expensify-protects-your-information/ opens without any issue
Actual Result:
Link broken and Error 1014 displayed
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Screen.Recording.2024-08-26.at.15.40.09.mov
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @adelekennedyThe text was updated successfully, but these errors were encountered: