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

CacheSetAdmin - cacheSetsToArchive visibility question #2353

Open
rapotek opened this issue May 9, 2022 · 2 comments
Open

CacheSetAdmin - cacheSetsToArchive visibility question #2353

rapotek opened this issue May 9, 2022 · 2 comments

Comments

@rapotek
Copy link
Contributor

rapotek commented May 9, 2022

In src/Controllers/Admin/CachesetAdminController the only requirement for cacheSetsToArchive method is for user to be logged in. That means any logged in user can view "Degraded GeoPaths", no OC Team member being is required.
Is it on purpose and correct or an oversight?

@stefopl
Copy link
Contributor

stefopl commented Jul 5, 2022

On my profile, I can see all the GeoPaths that I have completed. Some of them have been archived [12DŚBN] Trójmiasto 2021/2022 https://opencaching.pl/powerTrail.php?ptAction=showSerie&ptrail=1564. There is a link to them on the profile, but the page only says that Ta GeoŚcieżka jest niewidoczna.
In my opinion, archived GeoPath should be visible to at least the users who have completed it. For archived caches, it is still possible to display a description etc. so for geoPaths it should be similar

@rapotek
Copy link
Contributor Author

rapotek commented Jul 5, 2022

On my profile, I can see all the GeoPaths that I have completed. Some of them have been archived [12DŚBN] Trójmiasto 2021/2022 https://opencaching.pl/powerTrail.php?ptAction=showSerie&ptrail=1564. There is a link to them on the profile, but the page only says that Ta GeoŚcieżka jest niewidoczna. In my opinion, archived GeoPath should be visible to at least the users who have completed it. For archived caches, it is still possible to display a description etc. so for geoPaths it should be similar

Archived GeoPaths - yes. But here degraded GeoPaths are shown, i.e. these where available caches are not enough to complete a GeoPath, and the meaning of the list is to give information for OCTeam about GeoPaths which should be temporarily disabled. The location of the controller under Admin path suggest this should be used for administrative purposes only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants