Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Property types, part II #101

Merged
merged 8 commits into from
Feb 7, 2020
Merged

Conversation

vyacheslav-pushkin
Copy link
Collaborator

Related to #36

  • ✅ Render InputNumber for numeric fields:
long, decimal, int, double

Support for big numbers is tracked under separate ticket #99 as it depends on REST API changes

  • ✅ Add support for masked fields. Render masked field for uuid.
  • ✅ Fix displaying of Associations:
    • Values of many-to-many Associations are not displayed in DataTable
    • Associations should be shown in the same manner as in the Generic UI:
        - One-to-many Associations are not shown anymore at the owning side
        - One-to-one Associations are not shown anymore at the inverse side
  • ✅ Do not display byteArray (similar to Generic UI)

affects: @cuba-platform/react-ui, @cuba-platform/rest

related to #36
affects: @cuba-platform/react-ui, @cuba-platform/front-generator

related to #36
affects: @cuba-platform/react-ui, @cuba-platform/front-generator

- Values of many-to-many Associations are now displayed in DataTable
- Associations are now shown in the same manner as in the Generic UI:
  - One-to-many Associations are not shown anymore at the owning side
  - One-to-one Associations are not shown anymore at the inverse side

related to #36
affects: @cuba-platform/front-generator

related to #36
affects: @cuba-platform/react-ui

related to #36
@vyacheslav-pushkin vyacheslav-pushkin changed the base branch from master to develop February 6, 2020 14:30
@vadimbasko
Copy link
Contributor

vadimbasko commented Feb 7, 2020

Strongly recommend to write puppeteer tests for different PropertyTypes. All environment to easy test creation (ci run, login, samples) implemented in 'New cuba rest permission' issue #97 (hope it will be merged soon). Or I can separate it to different pull request, if you need it earlier.

@vadimbasko vadimbasko self-requested a review February 7, 2020 08:03
@vyacheslav-pushkin
Copy link
Collaborator Author

Strongly recommend to write puppeteer tests for different PropertyTypes. All environment to easy test creation (ci run, login, samples) implemented in 'New cuba rest permission' issue #97 (hope it will be merged soon). Or I can separate it to different pull request, if you need it earlier.

Agreed, I'll add puppeteer tests once #97 is merged. Created #105 to track that.

@vyacheslav-pushkin vyacheslav-pushkin merged commit f867e13 into develop Feb 7, 2020
@vyacheslav-pushkin vyacheslav-pushkin deleted the pv/feat/property-types2 branch February 7, 2020 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants