backport c027-2024.01.xx - Fix #10966 basic auth for services (#11067)#11070
backport c027-2024.01.xx - Fix #10966 basic auth for services (#11067)#11070allyoucanmap merged 11 commits intogeosolutions-it:c027-2024.01.xxfrom
Conversation
) * # This is a combination of 2 commits. Fix geosolutions-it#10966 basic auth for services Wip * Fix geosolutions-it#10966 basic auth for services * update config * fix jsdoc * fix jsdoc * fix tests * update review * clean up * fix tests * Update web/client/plugins/MetadataExplorer.jsx Co-authored-by: stefano bovio <stefano.bovio@geosolutionsgroup.com> * Update web/client/plugins/MetadataExplorer.jsx Co-authored-by: stefano bovio <stefano.bovio@geosolutionsgroup.com> --------- Co-authored-by: stefano bovio <stefano.bovio@geosolutionsgroup.com>
…utions-it#11077) * Fix geosolutions-it#10966 improved basic auth modal and logic * fix tests * created getAuthorizationBasic to harmonize code and tests
…modal (geosolutions-it#11082) * fix geosolutions-it#10966 some inconsistencies on the security popup modal * review changes (#7) * fix an error in widgets state * update tests * fix test and animation * fix test --------- Co-authored-by: stefano bovio <stefano.bovio@geosolutionsgroup.com>
|
|
||
| import React from 'react'; | ||
| import { FormControl as FormControlRB } from 'react-bootstrap'; | ||
| import withDebounceOnCallback from '../../../components/misc/enhancers/withDebounceOnCallback'; |
There was a problem hiding this comment.
This file can be removed it's not used
| ); | ||
| } | ||
|
|
||
| export default ConfirmDialog; |
There was a problem hiding this comment.
This file can be removed it's not used
| import localizedProps from '../../misc/enhancers/localizedProps'; | ||
| import {defaultPlaceholder, isValidURL} from "./MainFormUtils"; | ||
|
|
||
| import IconRB from '../../../plugins/ResourcesCatalog/components/Icon'; |
There was a problem hiding this comment.
In this case it's better to use directly Glyphicon so we can remove Icon from this PR
There was a problem hiding this comment.
this is a custom branch where we backport that feature, i would prefer to avoid to have too many differences from the master branch. i would skip this
| name, | ||
| className, | ||
| style | ||
| }) { |
There was a problem hiding this comment.
We can remove this component if we use Glyphicon instead in the other files. Then we can remove the ResourcesCatalog folder
| const FlexBox = forwardRef(({ | ||
| children, | ||
| className, | ||
| classNames = [], |
There was a problem hiding this comment.
missing less file with css rules
| * @prop {string} className custom class name | ||
| * @prop {object} style custom style object | ||
| */ | ||
| function Spinner({ |
There was a problem hiding this comment.
missing less file with css rules
| * @prop {string} textAlign one of `left`, `right` or `center` | ||
| * @prop {bool} strong if true it applies strong style | ||
| */ | ||
| const Text = ({ |
There was a problem hiding this comment.
missing less file with css rules
| borderTransparent, | ||
| ...props | ||
| }, ref) => { | ||
| return ( |
There was a problem hiding this comment.
missing less file with css rules
| @import "_base.less"; | ||
| @import "_brand-navbar.less"; | ||
| @import "_details-panel.less"; | ||
| @import "_footer.less"; | ||
| @import "_home-description.less"; | ||
| @import "_permissions.less"; | ||
| @import "_resource-card.less"; | ||
| @import "_resources-container.less"; | ||
| @import "_resources-panel-wrapper.less"; | ||
| @import "_resources-search.less"; | ||
| @import "_spinner.less"; | ||
| @import "_tags-manager.less"; |
There was a problem hiding this comment.
Currently the style is not applied because this file is not imported in the main mapstore.less theme.
Please include only what's needed that should be inside spinner.less and utility.less (please double check this)
51ca3d0
into
geosolutions-it:c027-2024.01.xx
Description
backport c027-2024.01.xx - Fix #10966 basic auth for services (#11067)