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

Incorporate previously reviewed changes (for 2.021.24) #3288

Merged
merged 684 commits into from
Jun 15, 2021
Merged

Incorporate previously reviewed changes (for 2.021.24) #3288

merged 684 commits into from
Jun 15, 2021

Conversation

jnm
Copy link
Member

@jnm jnm commented Jun 14, 2021

This, plus changes already in master, will become release 2.021.24.

PR Description Related Issues
#3281 Fix pre-release skip logic problem
Fixes missing skip logic value selector issue that was introduced on the beta branch but never released.
Fixes #3279
#3127 Feature: Library Locking UI
This includes the huge #3120 PR and we should wait for it to be merged before tackling this.
Fixes #3032
Fixes #2994
#3276 [Feature] Adds background-audio as a meta question type into formbuilder
Can now add a background-audio meta question to forms via the formbuilder sidebar:

image
Formbuilder side of things of #3193
Fixes #3275
#3278 Update sync_kobocat_xform command
#3226 Add sync_kobocat_form_media management command
Management command, sync_kobocat_form_media, created to allow for form-media from kobocat to be synced with KPI's replacement model.
closes #2675
#3273 Don't force form media's dropzone look everywhere
Revert global dropzone CSS changes, make changes specific to form media's dropzone.
#3272 Change Data side tabs order
"Table" tab is first now and is the default tab for "Data" section. "Reports" tab will now be second.
Fixes #3269
#3267 Make some of the dropdowns searchable
Selectors like "Country", "Sector" and few more are searchable now.
#3262 Form Media UI improvements
First iteration of form media got some upgrades:
- more spacing between components
- more clear UI elements (fonts, sizes of components)
#3265 Fix report styles tabs
Fixes a small bug with missing styles for report styles tabs.
#3264 Upgrade react-select dependency version
#2537 [Feature] Handle media files from KPI
Users can attach media files to their form and to preview them directly from KPI. Legacy interface is not used anymore.
Files are synchronized (asynchronously) on deployment only.
Closes #2673
Related to #2675
#3130 Alphabetization refactoring
Reorder methods of deployment back-end classes alphabetically for better readability.
Part of #2487
#3126 Add tests to asset deployments
This PR modifies the MockDeploymentBackend class to make it behave a little bit more like KobocatDeploymentBackend when deploying assets. It lets our unit tests cover more code that's used for asset deployment.
Part of #2487
#3251 Add the formbuilder message box from library locking
#3246 New names for gray colors
Introduced new future friendly names for gray color variables. Also updated the general hue and saturation of grays (went a bit more blue).
#3245 fix form view close button position
Fixed positioning of form view close button.
#3161 Create User Statistics Report
Created an admin view to track users usage for number of submissions, number of forms created and number of forms deployed
part of #3069
part of kobotoolbox/kobocat#703
#3209 [Library locking] Add locking to translation modal
Adds locking to the translation modal. Specifically, if a question, choice, or group label is locked, it's respective label in every translation is locked in the update translation modal

Known caveats: #3213
Fixes #3205


#3224 Iniitial table cleanup
Splitting the Table View related code into separate directory with some cleanup.
Preparing work for #3199 and #3198
Part of #3225
#3229 Fix hiding columns
Fix hiding Submission links column. Never hide checkboxes column.
Fixes #3227
Fixes #3228
#3223 Fixed GenericForeignKey errors with shadow models
Overloads GenericField class to check for KobocatContentType objects instead of ContentType.
Shadow models are reading data from KoBoCAT databases, so related objects should get content types from the same DB.
Content types can differ between databases
Fixes issue found here #3161 (comment)
#3064 cleaned up translation strings
Removed the _() wrapping used to identify which strings should be translated from any of the strings which do not appear to the front end users.

Added a shell script to automate creating PO files and pushing them to Transifex.
No related issues
#3220 Linter arrow functions improvement
Improve linter configuration to allow more compact arrow functions.
#3185 Split reports code
Split Reports code into multiple files to make maintaining it easier.
#3217 Add prettier config
Add prettier configuration file for better development.
#3216 Fix AnonymousUser listed permissions on public asset
Now included in master branch by #3256; can omit from release notes
closes #3215
#3211 Better light button disabled styles
More (visually) distinct disabled state for light buttons.
#3184 Missing collection link handling
Whenever collection method link is missing from endponit (e.g. due to Enketo error), show a warning information instead of broken button.
#3189 Create project button fix
Allow creating projects from any viewable templates (previously user needed edit permissions).
Fixes #3162
#3165 Don't show last asset after it's removed from asset list
Adds a hack fix that removes last asset from list view after it's been deleted
Fixes #3163
#3152 Add non-owner self removal to form landing
Add the non-owner self removal option introduced in #2984 to the actions available in form landing (to compliment where a user would expect the sharing options to be for an owned form):
image
Fixes #3151
#3111 Automated pushing translation strings to Transifex
Automated the task of pushing an updated list of translation strings
This is to keep our translations as accurate as possible and all strings will be pushed to Transifex every time a PR to the Master branch is made.
Blocked by #3064
#3118 Link export with export-settings
Include export-settings name in the export data object if provided.
backend for #3078
blocked by kobotoolbox/formpack#240
#3082 Library Locking
Backend support for library locking features, integrating the new formpack changes.
closes #3005
blocked by kobotoolbox/formpack#236
#3146 Delete entire group functionality
Delete entire group functionality. Now whenever use clicks the delete group button, a modal will be displayed to choose wheter they want to simply ungroup or delete everything.
Fixes #3132

Needed for #3032 (will merge it there before this is reviewed)
#3155 Fix to allow non-owners with manage_asset permission to remove other non-owners' permissions
A fix to allow non-owners with manage_asset permissions to manage all permissions of a form including removing all permissions of other non-owners.
closes #3154
#3141 Post coffeelint config fixes
Fixing a few coffeescript lines that were raisng compiler errors for linter.
#3143 3140 new view all icon
A better new icon for "Expand / collapse questions" button.
Fixes #3140
#3136 Coffeelint config
Added coffee code linting configuration file. Almost all the rules added are set to "warn" level. This is going to be helpful for #3018 to clean up the code before running scripts.
Part of #3018
#3120 Update icons, get rid of FontAwesome
All of our icons now are self designed, we stopped using FontAwesome. Some of the icons got updated. Also introduced consistent loading spinner with message.
Fixes #2869
Part of #3032
#3129 Fix not redirecting to form list after deleting project
Fixed bug where if user deleted a project they would not be redirected back to the form list
#2984 Add dropdown menu item that lets non-owners remove their own permissions
Adds a dropdown menu item on the asset list page that lets a non-owner remove forms that are shared with them from their asset list.
Front end of #2940
#3076 Better routes checking
Alternative solution to react router's isActive for checking wheter a route is your current route. Needed for non-components.
Started because of #3012
#3077 Add comment about _status field
Add comments for developers describing the _status field, which comes from KoBoCAT
#3110 Project downloads settings not saving properly bug fix
The selected defined export settings should be cleared on any form changes. This PR fixes the last few inputs that weren't clearing it properly.

Also includes disabling "select all" (rows) when all are selected and "unselect all" (rows) when none are selected.
Fixes #3109
#3112 Move surveyCompanion store to separate file
This is small code refactor change, nothing user oriented. I moved one of the stores to a separate file to avoid potential circular import warning.
Part of #3032
#2925 Reworked basic form media UI with actions
Implemented actions to help serve the form media modal. Improves error handling and extensibility.
#2695
#2695 Basic form media uploading modal
Very basic UI made to replace the KC iframe in settings/media. Still basic and needs ui design but functionality described below has been achieved:

- [x] Upload single files
- [x] Upload multiple files (I only noticed this was possible a few days ago ! )
- [x] Add media URLs (pending back-end support)
- [x] View current files
- [x] View current URLs
- [x] Delete files
- [x] Remove URLs
- [x] Remove owner only access to form media page (#2647, #2657)

URL functionality pending back-end support: https://www.flowdock.com/app/kobotoolbox/kobo/threads/78V2a4RfL2rIlTdbZuuFkplFkc2

Functionality so far:
basic-form-media-ui
Fixes #2673, #2647

magicznyleszek and others added 30 commits May 1, 2021 00:18
…rt-settings

Link export with export-settings
Automated pushing translation strings to Transifex
joshuaberetta and others added 26 commits June 10, 2021 18:14
Don't force form media's `dropzone` look everywhere
Add `sync_kobocat_form_media` management command
 * Since the feature is new on all collects, specify version number
 * wording: "functionality only works in" -> "functionality available in"
# Conflicts:
#	jsapp/js/components/formLanding.es6
#	jsapp/js/components/formSummary.es6
#	jsapp/js/components/formViewTabs.es6
#	jsapp/js/components/header.es6
#	jsapp/js/components/library/ownedCollectionsStore.es6
#	jsapp/js/components/reportViewItem.es6
#	jsapp/js/constants.es6
#	jsapp/js/utils.es6
#	jsapp/scss/components/_kobo.drawer.scss
#	jsapp/scss/components/_kobo.form-view.scss
#	locale
[Feature] Adds `background-audio` as a meta question type into formbuilder
Fix outdated color names and obvious merge conflicts that I didn't
notice until front-end compilation failed
@jnm jnm self-assigned this Jun 15, 2021
@jnm jnm merged commit ce65f64 into master Jun 15, 2021
jdugh pushed a commit to geohyd/kpi that referenced this pull request Nov 21, 2022
Incorporate previously reviewed changes (for 2.021.24)
jdugh pushed a commit to geohyd/kpi that referenced this pull request Nov 13, 2023
Incorporate previously reviewed changes (for 2.021.24)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment