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

[Feature] Column filtering for basic case management #3117

Merged
merged 47 commits into from May 31, 2021

Conversation

duvld
Copy link
Member

@duvld duvld commented Mar 30, 2021

Merge and test #3057 first!

Description

Extends base UI for exporting/importing data across connected projects with column (question) selection

Related issues

Column filtering of #2767

Conflicts:
	jsapp/js/actions.es6
	jsapp/js/components/modalForms/connectProjects.es6
and some clean up
Conflicts:
	jsapp/js/components/modalForms/connectProjects.es6
Clean up with PR Changes of #3057 in mind

 Conflicts:
	jsapp/js/actions.es6
	jsapp/js/components/formSubScreens.es6
	jsapp/js/components/modalForms/connectProjects.es6
	jsapp/scss/components/_kobo.connect-projects.scss
	jsapp/scss/main.scss
@duvld duvld requested a review from joshuaberetta April 1, 2021 19:28
}

componentWillUnmount() {
this.unlisteners.forEach((clb) => {clb();});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe rather this?

this.unlisteners.forEach(clb => clb());

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, looks cleaner 👍

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually everywhere else does it like this, maybe a question for @magicznyleszek :D

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah let's do it like that 👍 I have personal issue with that due to too much coffeescripting :P but it's not a logical argument :)

jsapp/js/actions/datashare.es6 Outdated Show resolved Hide resolved
jsapp/js/constants.es6 Show resolved Hide resolved
jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
- use const/let properly
- use safer/consistent object access
- alertify error handling for all requests
- remove unused imports
- fix css class mismatch
- don't mutate state object directly for fieldsErrors
@Ig-Rebollo
Copy link

Ig-Rebollo commented Apr 12, 2021

Sorry for the delay @duvld and @joshuaberetta

Here are some mockups on how to implement these suggestions.

image

As you can see I included a counter to give users confirmation of when the questions have been actually shared (if they click on the 'share' button without selecting any questions, maybe a notification saying 'no questions have been selected' could appear.

The 'loading' could be part of the button itself

image

image

Note I also changed the wording for the description text on top of the table. I liked @joshuaberetta option better.

Let me know what you think!

jsapp/js/components/modalForms/connectProjects.es6 Outdated Show resolved Hide resolved
jsapp/scss/components/_kobo.form-view.scss Outdated Show resolved Hide resolved
jsapp/js/utils.es6 Show resolved Hide resolved
@magicznyleszek magicznyleszek merged commit 59e8489 into 2767-basic-case-management May 31, 2021
@magicznyleszek magicznyleszek deleted the 2767-column-filtering branch May 31, 2021 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants