updates as per issue - Githubissues

DigitalCommons / land-explorer-front-end

React app for the Land Explorer front end

http://landexplorer.cc

GNU Affero General Public License v3.0

1 stars 0 forks source link

updates as per issue #322

Closed ms0ur1s closed 2 months ago

ms0ur1s commented 2 months ago

What? Why?

Update user menu, remove superfluous links, update donate URL.

What should we test?

Check the donate link directs to: https://opencollective.com/digitalcommonscoop
Check the My Maps link open the same modal as 'Open' on the map menu

rogup commented 2 months ago

@ms0ur1s this looks good :) One extra thing to do is to remove some of the old unused code, now that we don't need separate My Maps and Shared Maps modals. I think this is just

the MyMapsModal and MySharedMapsModal components
the myMaps and mySharedMaps objects that exist in the ModalReducer

Feel free to just merge it after this, everything else is great!

ms0ur1s commented 2 months ago

@ms0ur1s this looks good :) One extra thing to do is to remove some of the old unused code, now that we don't need separate My Maps and Shared Maps modals. I think this is just
* the `MyMapsModal` and `MySharedMapsModal` components

* the `myMaps` and `mySharedMaps` objects that exist in the ModalReducer
Feel free to just merge it after this, everything else is great!

Thanks @rogup, I'll have a look at these now and make the changes.