Skip to content

#1977: Move group setting to management panel#2024

Merged
allyoucanmap merged 6 commits intoGeoNode:masterfrom
dsuren1:#1977_group_owner
Jun 19, 2025
Merged

#1977: Move group setting to management panel#2024
allyoucanmap merged 6 commits intoGeoNode:masterfrom
dsuren1:#1977_group_owner

Conversation

@dsuren1
Copy link
Copy Markdown

@dsuren1 dsuren1 commented Apr 15, 2025

Description

The PR moves the group to Management panel.

Update: This PR is dependent on permissions PR. When #2017 is merged, this PR will be aligned to avoid any merge conflicts

Note

For testing locally, the backend branch in geonode is ISSUE_12996. Backend PR

Issue

Screenshot

image

@dsuren1 dsuren1 added this to the 5.0.0 milestone Apr 15, 2025
@dsuren1 dsuren1 requested a review from allyoucanmap April 15, 2025 10:49
@dsuren1 dsuren1 self-assigned this Apr 15, 2025
@dsuren1 dsuren1 linked an issue Apr 15, 2025 that may be closed by this pull request
@dsuren1 dsuren1 force-pushed the #1977_group_owner branch from 9f2e3c1 to f0c756e Compare May 9, 2025 10:11
@dsuren1 dsuren1 changed the base branch from geonode-home to master May 9, 2025 10:11
dsuren1 added 4 commits June 16, 2025 19:18
(cherry picked from commit f19ea4a0f62a0ca8f446af653572872fcf1211d4)
@dsuren1
Copy link
Copy Markdown
Author

dsuren1 commented Jun 17, 2025

@allyoucanmap This PR is ready for review. Group modification has been added to the management panel, and the owner field has been removed in line with the latest requirements for this issue

@dsuren1 dsuren1 changed the title #1977: Move group and owner setting to management panel #1977: Move group setting to management panel Jun 17, 2025
Copy link
Copy Markdown

@allyoucanmap allyoucanmap left a comment

Choose a reason for hiding this comment

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

There are changes inside the submodule, shouldn't be aligned with master?

function DetailsSettings({ resource, onChange }) {
return (
<FlexBox column gap="md" className="gn-details-settings _padding-tb-md">
<FlexBox.Fill gap="xs" className="_padding-b-xs">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we need specific permission to visualize this dropdown?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Not that I can remember of and not specifically asked in the requirement too. Only that it needs to be disabled when permission is missing change_resourcebase

@dsuren1
Copy link
Copy Markdown
Author

dsuren1 commented Jun 19, 2025

There are changes inside the submodule, shouldn't be aligned with master?

geonode-mapstore-client's master is aligned with 2025.01.xx of MS. So I have just pulled in the latest changes of the stable branch from MS

@dsuren1 dsuren1 requested a review from allyoucanmap June 19, 2025 09:03
@allyoucanmap allyoucanmap merged commit 8f2a55d into GeoNode:master Jun 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move group and owner setting to management panel

2 participants