Skip to content

New Setting to make Profile Public, new UserSettings table#314

Open
xlcrr wants to merge 9 commits intostagingfrom
user-dashboard/make-public
Open

New Setting to make Profile Public, new UserSettings table#314
xlcrr wants to merge 9 commits intostagingfrom
user-dashboard/make-public

Conversation

@xlcrr
Copy link
Copy Markdown
Member

@xlcrr xlcrr commented Aug 14, 2021

We are changing the design of the users profile page

and introduce a new setting to make it public or private.

If the setting is public, anyone can visit openlittermap.com/username to see the users profile and interact with their data.

The user can show their rank, total images, total litter and total categories

We have another setting to view map + download data.

If we give the viewer the ability to view the map data, we are essentially giving them the ability to download the data so I think we need to group these together to be clear about it

Any comments or suggestions?

Screenshot 2021-08-28 at 23 36 48

Screenshot 2021-08-28 at 23 39 08

@xlcrr xlcrr self-assigned this Aug 14, 2021
@xlcrr xlcrr added enhancement New feature or request work in progress labels Aug 14, 2021
@xlcrr xlcrr added the help wanted Extra attention is needed label Aug 28, 2021
@xlcrr xlcrr requested a review from GeniJaho August 28, 2021 22:37
# Conflicts:
#	.idea/olm.iml
#	.idea/php.xml
#	app/Http/Controllers/User/ProfileController.php
#	composer.lock
#	public/js/app.js
#	public/mix-manifest.json
#	resources/js/components/Profile/middle/ProfileMap.vue
#	resources/js/store/modules/user/index.js
#	resources/js/store/modules/user/mutations.js
@GeniJaho
Copy link
Copy Markdown
Collaborator

I propose we use a package to handle user settings. This is both for new settings added and migrating the old ones. Having them as database columns makes it pretty hard to update them.
I propose we use this package https://github.com/DarkGhostHunter/Laraconfig which I think is more suitable for our use-case; another option is https://github.com/glorand/laravel-model-settings.

@xlcrr xlcrr added the parked label Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed parked work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants