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

list abandoned zones #1284

Merged
merged 15 commits into from
Sep 12, 2023
Merged

Conversation

Jay07GIT
Copy link
Member

@Jay07GIT Jay07GIT commented Jul 24, 2023

Fixes #617

Changes in this pull request:

  • Added API for deleted zones history (zones/deleted/changes)
  • Made Portal changes to view deleted zones history.
  • Removed Cascade from zone_access table. We use this on comparing zone_change table .
  • Added zone_status column in zone_access table to identify the deleted zones and used the same in deleting the group.

@arpit4ever arpit4ever modified the milestones: v0.21.0, v0.19.1 Jul 24, 2023
@Jay07GIT Jay07GIT force-pushed the deleted_all_zones_history branch 2 times, most recently from 99b2d11 to d494d0f Compare July 25, 2023 14:18
@Jay07GIT Jay07GIT self-assigned this Jul 25, 2023
@Jay07GIT Jay07GIT added kind/enhancement A modification or enhancement to existing functionality area/portal Items relating to the Portal area/api Items relating to the API area/db Items relating to data or the database labels Jul 25, 2023
arpit4ever
arpit4ever previously approved these changes Aug 15, 2023
Copy link
Contributor

@arpit4ever arpit4ever left a comment

Choose a reason for hiding this comment

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

LGTM
Tested (GUI & API) functionality locally.

Copy link
Member

@nspadaccino nspadaccino left a comment

Choose a reason for hiding this comment

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

I'm seeing a bug in the zones view.
If I log in as one user, create a group, and connect a zone, then log in with another user not in that group, I am able to see the zone in My Zones, and I am able to click on it. When I click on it, I get a 403 error (since that user doesn't have permission to view that private zone).
Screenshot 2023-08-16 at 11 19 09 AM
Other groups' private zones shouldn't appear in My Zones tab

@nspadaccino
Copy link
Member

Also, it seems like wildcard search isn't working correctly for abandoned zones.
In My Zones, I can search for "o*" and see the zone "ok". But when I search for "o*" in abandoned zones, I get no results.
Screenshot 2023-08-16 at 11 30 24 AM
Screenshot 2023-08-16 at 11 30 32 AM
Screenshot 2023-08-16 at 11 30 40 AM
Screenshot 2023-08-16 at 11 30 47 AM

@arpit4ever arpit4ever modified the milestones: v0.19.1, v0.19.2 Aug 18, 2023
@Jay07GIT
Copy link
Member Author

Also, it seems like wildcard search isn't working correctly for abandoned zones. In My Zones, I can search for "o*" and see the zone "ok". But when I search for "o*" in abandoned zones, I get no results. Screenshot 2023-08-16 at 11 30 24 AM Screenshot 2023-08-16 at 11 30 32 AM Screenshot 2023-08-16 at 11 30 40 AM Screenshot 2023-08-16 at 11 30 47 AM

Yes. its right . Since we used scala filter we dont need wildcard for seach. we can jus give "o" to search ok. zones.

@Jay07GIT
Copy link
Member Author

I'm seeing a bug in the zones view. If I log in as one user, create a group, and connect a zone, then log in with another user not in that group, I am able to see the zone in My Zones, and I am able to click on it. When I click on it, I get a 403 error (since that user doesn't have permission to view that private zone). Screenshot 2023-08-16 at 11 19 09 AM Other groups' private zones shouldn't appear in My Zones tab

Oh yes. Updated the code. Now its working fine.

Copy link
Contributor

@arpit4ever arpit4ever left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@nspadaccino nspadaccino left a comment

Choose a reason for hiding this comment

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

looks good

@arpit4ever arpit4ever merged commit 894ddf3 into vinyldns:master Sep 12, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Items relating to the API area/db Items relating to data or the database area/portal Items relating to the Portal kind/enhancement A modification or enhancement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No way to view zone change history on deleted zone
4 participants