Datahub merge requestshttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests2023-02-01T09:01:20+01:00https://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/669Resolve "Released logbook can be updated"2023-02-01T09:01:20+01:00Marjolaine BodinResolve "Released logbook can be updated"Closes #655Closes #655Marjolaine BodinMarjolaine Bodinhttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/631Resolve "Page My selection broken"2022-08-26T11:28:08+02:00Marjolaine BodinResolve "Page My selection broken"Closes #615Closes #615Marjolaine BodinMarjolaine Bodinhttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/570Resolve "Photo upload does not work"2021-11-30T15:07:51+01:00Alejandro De Maria AntolinosResolve "Photo upload does not work"Closes #551Closes #551Alejandro De Maria AntolinosAlejandro De Maria Antolinoshttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/538Resolve "Search problem with parcels"2021-07-23T10:43:13+02:00Alejandro De Maria AntolinosResolve "Search problem with parcels"Closes #520Closes #520Alejandro De Maria AntolinosAlejandro De Maria Antolinoshttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/506Resolve: "User can see manager menu"2021-05-06T06:40:30+02:00Marjolaine BodinResolve: "User can see manager menu"Closes #494
* move `Beamline permissions` under `My Beamlines`
* move `My Beamlines` to the left of `Manager`
* menu `Manager` only for Administrator
* fix function name: `ManagerMenu` in `BeamlinesMenu.js`: very strange because in `Me...Closes #494
* move `Beamline permissions` under `My Beamlines`
* move `My Beamlines` to the left of `Manager`
* menu `Manager` only for Administrator
* fix function name: `ManagerMenu` in `BeamlinesMenu.js`: very strange because in `Menu.js` the import was already:`import BeamlinesMenu from './BeamlinesMenu';` whereas the function was `ManagerMenu`. I am wondering how it worked.Alejandro De Maria AntolinosAlejandro De Maria Antolinoshttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/505Revert administrator filter for menu manager2021-05-04T15:40:41+02:00Marjolaine BodinRevert administrator filter for menu manager#494#494Marjolaine BodinMarjolaine Bodinhttps://gitlab.esrf.fr/icat/E-DataPortal/-/merge_requests/259Add and display addresses2020-06-23T10:58:31+02:00Loic HuderAdd and display addresses- Adds a tab "Manage Adresses" that displays all addresses of a user
![image](/uploads/44bec4b8a2aade64770702c28b7495aa/image.png)
- Adds a form to add a new address to a proposal. The form is validated using [react-hook-form](https://...- Adds a tab "Manage Adresses" that displays all addresses of a user
![image](/uploads/44bec4b8a2aade64770702c28b7495aa/image.png)
- Adds a form to add a new address to a proposal. The form is validated using [react-hook-form](https://react-hook-form.com/) capabilities.
![image](/uploads/e25a3e060e83991a4c74e367aaa91d2c/image.png)
This MR is to discuss first the implementation of these basic functionalities before the full-fledged implementation of #255.
Points I would like to discuss:
- <s>Integration in the UI and layout in general</s>
> **Used `app__inner` as suggested by @bocciare**
- <s>Use of Redux ? I didn't use Redux state at all so I will be happy to have any pointers on that.</s>
> **Done.**
- <s>Request implementations. I put them in a dedicated `api.js` file but you might want to add the URI to your "catalogue".</s>
> **Added to `api/icatPlus.js`**
- <s>Author/CreatedBy field. Is this needed ? If yes, how can I get the name from the user id ?</s>
> **Hidden for now**
- <s>New routes. I put the form at `addresses/new` and the address table at `addresses/`. Is this fine by you ?</s>
> **Same pattern as `shipments`**