mmoio commented 3 years ago

Overview

Please include a summary of your contribution.

Related PR in world-scribe-2-server

[x] I have verified that all 3 components of the app are active after running npm start: the React server at http://localhost:490001, the backend server at http://localhost:49000, and the World Scribe 2 application window.
[x] I have verified that communication between the application and the backend server is still working (e.g. by verifying that the application is making HTTP requests and receiving responses).

[x] I have verified that all affected components are visible and responsive when the application window is maximized.
[x] Where applicable, I display informative error messages to the user.

[x] I have verified that ALL settings are saved properly between app sessions.
[x] For new settings: I have verified that my new setting(s) appear on both the Initial Setup page and the App Settings page.

mmoio commented 3 years ago

Where error display logic should be, I have left a TODO comment like I've seen in other part of the code

mmoio commented 3 years ago

Is possible to rename items in theirs respective list or clicking on the item title bar

MarquisLP commented 3 years ago

Thank you for your contribution! I will review this as soon as I can.

MarquisLP commented 3 years ago

Awesome, the UI looks perfect now! 🎉

The last thing to address before this PR gets merged is the comment I left on the backend PR in world-scribe-2-server. (Not sure if you've seen it yet.)

mmoio commented 3 years ago

I've changed the routes on the backend to /name and updated the frontend accordingly