marmelab / ra-supabase

Supabase adapter for react-admin, the frontend framework for building admin applications on top of REST/GraphQL services.
MIT License
156 stars 27 forks source link

Update demo #59

Closed erwanMarmelab closed 4 months ago

erwanMarmelab commented 5 months ago

Problem

React-Admin CRM demo was updated

Solution

Update ra-supabase demo

slax57 commented 5 months ago

Found a couple more issues while testing: :grimacing:

  1. Got the following warning:

    Warning: validateDOMNesting(...): <div> cannot appear as a descendant of <p>.
    at div
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at Box3 (http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:7450:19)
    at LinearProgress (http://localhost:8000/node_modules/.vite/deps/chunk-PXPP5S64.js?v=d562344d:4705:18)
    at ReferenceFieldView (http://localhost:8000/node_modules/.vite/deps/chunk-PXPP5S64.js?v=d562344d:92793:24)
    at ResourceContextProvider (http://localhost:8000/node_modules/.vite/deps/chunk-IT6VC33E.js?v=d562344d:13043:21)
    at NonEmptyReferenceField (http://localhost:8000/node_modules/.vite/deps/chunk-PXPP5S64.js?v=d562344d:92778:23)
    at ReferenceField (http://localhost:8000/node_modules/.vite/deps/chunk-PXPP5S64.js?v=d562344d:92752:22)
    at p
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at Typography2 (http://localhost:8000/node_modules/.vite/deps/chunk-UDCJ5W4P.js?v=d562344d:4213:22)
    at div
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at ListItemText2 (http://localhost:8000/node_modules/.vite/deps/chunk-UDCJ5W4P.js?v=d562344d:15863:17)
    at a
    at LinkWithRef (http://localhost:8000/node_modules/.vite/deps/chunk-6SAAVCAB.js?v=d562344d:291:5)
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at ButtonBase2 (http://localhost:8000/node_modules/.vite/deps/chunk-UDCJ5W4P.js?v=d562344d:3264:17)
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at li
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at ListItem2 (http://localhost:8000/node_modules/.vite/deps/chunk-UDCJ5W4P.js?v=d562344d:15483:17)
    at RecordContextProvider (http://localhost:8000/node_modules/.vite/deps/chunk-IT6VC33E.js?v=d562344d:6251:21)
    at ul
    at http://localhost:8000/node_modules/.vite/deps/chunk-ZOST5TTG.js?v=d562344d:4676:51
    at List2 (http://localhost:8000/node_modules/.vite/deps/chunk-UDCJ5W4P.js?v=d562344d:15046:17)
    at ContactListContent (
  2. In ContactList filter, got the following error

    column contacts.q does not exist
  3. In ContactCreate, when selecting a company, if I type something in the field I get the following error: 400 Bad Request http://localhost:54321/rest/v1/companies?offset=0&limit=25&q=eq.cc&order=id.desc

    "message": "column companies.q does not exist"
  4. In ContactNoteCreate The status input does not have a default value nor validation, but it is required according to supabase Also, we get the following warning in the console:

    Warning: A component is changing an uncontrolled input to be controlled.
  5. After adding a contact to a company, when returning to the company page, we can see last activity about 2 hours ago although the contact was just added. There may be an issue regarding the dates management.

  6. CompanyShow does not include Contacts and Deals tabs. company.nb_contacts and company.nb_deals appear to be missing in database.

  7. CompanyList filter throws an error.

  8. Adding a new deal_note triggers a call to update the deal that ends up in error:

    Column 'last_seen' of relation 'deals' does not exist
  9. DealList filter only filters on Deal name. It should do a full text search instead.

slax57 commented 4 months ago

All issues found in the demo should be fixed now! :white_check_mark: