Skip to content
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

[HOLD for Payment 2024-09-12][$62.50] Add new sub-header to More features page #48023

Closed
joekaufmanexpensify opened this issue Aug 26, 2024 · 30 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item.

Comments

@joekaufmanexpensify
Copy link
Contributor

joekaufmanexpensify commented Aug 26, 2024

Problem

Coming from here, there are concerns that the current More features page on the workspace editor doesn't make it sufficiently clear that enabling a toggle often simply adds a feature to the workspace editor to configure the feature, rather than actually turning it on. This could impact feature adoption, or drive support volume.

Solution

Add a new sub-header directly under More features header on this page of the workspace, that clarifies that toggling something on simply adds the feature to the workspace editor.

Sub-header copy should be: Use the toggles below to enable more features as you grow. Each feature will appear in the navigation menu for further customization.

It should go directly under More features, similar to how the Categories page shown below has a sub-header.

2024-08-26_15-06-06 2024-08-26_15-06-06

cc @heyjennahay @jamesdeanexpensify @JmillsExpensify @shawnborton @dannymcclain

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0182dc1eaaa54e652c
  • Upwork Job ID: 1828148191299361076
  • Last Price Increase: 2024-08-26
  • Automatic offers:
    • rayane-djouah | Reviewer | 103742172
    • Krishna2323 | Contributor | 103742174
Issue OwnerCurrent Issue Owner: @Krishna2323
@joekaufmanexpensify joekaufmanexpensify added External Added to denote the issue can be worked on by a contributor Daily KSv2 NewFeature Something to build that is a new item. labels Aug 26, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to @adelekennedy (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to Design team member for new feature review - @dannymcclain (NewFeature)

@joekaufmanexpensify joekaufmanexpensify added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. and removed Weekly KSv2 labels Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Current assignee @adelekennedy is eligible for the Bug assigner, not assigning anyone new.

@joekaufmanexpensify joekaufmanexpensify added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0182dc1eaaa54e652c

@melvin-bot melvin-bot bot changed the title Add new sub-header to More features page [$250] Add new sub-header to More features page Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @rayane-djouah (External)

Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External)

@joekaufmanexpensify joekaufmanexpensify changed the title [$250] Add new sub-header to More features page [$62.50] Add new sub-header to More features page Aug 26, 2024
@joekaufmanexpensify
Copy link
Contributor Author

Adjusted price to $62.50, since this is a very, very minor change. Automation didn't work with Upwork, so adjusted the price manually there too

@joekaufmanexpensify joekaufmanexpensify removed the Bug Something is broken. Auto assigns a BugZero manager. label Aug 26, 2024
@Krishna2323
Copy link
Contributor

Krishna2323 commented Aug 26, 2024

Edited by proposal-police: This proposal was edited at 2024-08-26 19:26:30 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

Add new sub-header to More features page

What is the root cause of that problem?

Improvement

What changes do you think we should make in order to solve the problem?

Below this line we need to add <Text style={[styles.pl5, styles.mb4, styles.mt3, styles.textSupporting]}>{translate('workspace.moreFeatures.subtitle')}</Text>, the translation will be different and will be added in workspace.moreFeatures object in en.ts and es.ts file.

We can also apply maxWidth for the text:

                <Text style={[styles.pl5, styles.mb4, styles.mt3, styles.textSupporting, shouldUseNarrowLayout ? styles.workspaceSectionMobile : styles.workspaceSection]}>
                    {translate('workspace.moreFeatures.subtitle')}
                </Text>

What alternative solutions did you explore? (Optional)


Result

image

@daledah
Copy link
Contributor

daledah commented Aug 26, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

Add new sub-header to More features page

What is the root cause of that problem?

New Feature

What changes do you think we should make in order to solve the problem?

  1. We need to add new translation workspace.moreFeatures.subtitle

  2. Create a new function and use this in here

    const getHeaderText = () => (
        <View style={[styles.ph5, styles.pb5, styles.pt3]}>
            <Text style={[styles.textNormal, styles.colorMuted]}>{translate('workspace.moreFeatures.subtitle')}</Text>
        </View>
    );

What alternative solutions did you explore? (Optional)

@melvin-bot melvin-bot bot added the Overdue label Aug 29, 2024
@rayane-djouah
Copy link
Contributor

Friendly bump @AndrewGable on #48023 (comment)

@melvin-bot melvin-bot bot removed the Overdue label Aug 29, 2024
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 29, 2024
Copy link

melvin-bot bot commented Aug 29, 2024

📣 @rayane-djouah 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

Copy link

melvin-bot bot commented Aug 29, 2024

📣 @Krishna2323 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Sep 2, 2024
@Krishna2323
Copy link
Contributor

@joekaufmanexpensify, could you please provide the spanish translation? Or should we ask on slack?

@rayane-djouah, PR ready for review ^

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Sep 2, 2024
Copy link

melvin-bot bot commented Sep 4, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@rayane-djouah
Copy link
Contributor

This ^ was a very small issue and was addressed in a follow-up PR: #48573

@rayane-djouah
Copy link
Contributor

Note

The production deploy automation failed: This should be on [HOLD for Payment 2024-09-12] according to #48515 production deploy checklist, confirmed in #48389 (comment)

@adelekennedy adelekennedy changed the title [$62.50] Add new sub-header to More features page [HOLD for Payment 2024-09-12][$62.50] Add new sub-header to More features page Sep 9, 2024
@rayane-djouah
Copy link
Contributor

@adelekennedy This is due payment

@adelekennedy
Copy link

adelekennedy commented Sep 18, 2024

Payouts due:

Upwork job is here.

@adelekennedy adelekennedy added Daily KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 Reviewing Has a PR in review labels Sep 18, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 20, 2024
@adelekennedy
Copy link

offer still pending @Krishna2323

@Krishna2323
Copy link
Contributor

Sorry for delay @adelekennedy, offer accepted.

@melvin-bot melvin-bot bot removed the Overdue label Sep 23, 2024
@github-project-automation github-project-automation bot moved this from Release 2.5: SuiteWorld (Sept 9th) to Done in [#whatsnext] #wave-collect Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item.
Projects
No open projects
Status: Done
Development

No branches or pull requests

16 participants