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

editor: adapt JSON schema form options #1647

Merged
merged 1 commit into from
Feb 11, 2021
Merged

editor: adapt JSON schema form options #1647

merged 1 commit into from
Feb 11, 2021

Conversation

jma
Copy link
Contributor

@jma jma commented Jan 27, 2021

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

Dependencies

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Cypress tests successful?

@jma jma force-pushed the maj-new-editor branch 3 times, most recently from 89dc517 to 2a12c77 Compare January 27, 2021 14:48
@jma jma changed the title in progress editor: adapt JSONSchema form options Jan 27, 2021
@jma jma force-pushed the maj-new-editor branch 3 times, most recently from cd0a2c3 to 00270f9 Compare January 28, 2021 11:47
@jma jma marked this pull request as ready for review January 28, 2021 11:48
@jma jma requested review from zannkukai and BadrAly January 28, 2021 15:26
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the commit message:

  • Separate JSON from schema in the title.
  • url to URL.

@iGormilhit iGormilhit added f: data About data model, importation, transformation, exportation of data, specific for bibliographic data f: editor Concerns editor based on JSON schema AND custom editor labels Feb 1, 2021
@iGormilhit iGormilhit added this to the v1.1.0 milestone Feb 1, 2021
@jma jma force-pushed the maj-new-editor branch 2 times, most recently from 53b76af to 87cbe2b Compare February 1, 2021 13:12
@jma jma requested a review from iGormilhit February 1, 2021 13:12
@iGormilhit iGormilhit changed the title editor: adapt JSONSchema form options editor: adapt JSON schema form options Feb 2, 2021
@JoelleDosimont
Copy link
Contributor

Seems all right to me, except for #1423 where the term "agent" remains unchanged.

@jma
Copy link
Contributor Author

jma commented Feb 10, 2021

Seems all right to me, except for #1423 where the term "agent" remains unchanged.

This will be fixed when the translations will be done.

@JoelleDosimont
Copy link
Contributor

In the holding editor, when adding a fields "Notes" it doesn't appear with a + sign like in the item editor. See the screenshot :
image

@pronguen
Copy link
Contributor

pronguen commented Feb 10, 2021

I tested it and everything works! The link in the autocomplete is a great improvement, and will have a real impact on the end user/cataloguer.

Two remarks:

  1. Display a hyper text link when adding a $ref link to another resource in the editor #1425 is not really solved (except for MEF link)
    • the initial goal was not for MEF link but for $ref in general.
    • the other $ref are not displayed as links. Examples:
      • Resource "acq_order_lines": field document
      • Resource "document": field partOf.document
    • There is a confusion: in some cases, the link is intern to RERO ILS, in some other (MEF link) it is either to MEF or to IdRef/GND.
    • Can you take out this issue from the PR and put the label "needs info"?
  2. Contributor autocomplete in document editor does not ignore diacritics #1462 seems to be solved. If yes, can you include it as solved in the PR?

@iGormilhit
Copy link

iGormilhit commented Feb 11, 2021

@jma @pronguen Is there a confusion between #1598 and #1425 ?

⚠️ The list of closed issues is not the same in the PR comment or in the commit message. Maybe it's on purpose, just be sure to close the right one with the commit message.

* Adapts the JSON schema form options for a new ng-core editor version.
* Changes the schemaform URL to schemas to be more compliant with other
  RERO application.
* Closes rero#1426.
* Closes rero#1423.
* Closes rero#1366.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma
Copy link
Contributor Author

jma commented Feb 11, 2021

@jma @pronguen Is there a confusion between #1598 and #1425 ?

⚠️ The list of closed issues is not the same in the PR comment or in the commit message. Maybe it's on purpose, just be sure to close the right one with the commit message.

This is a summary of issues for all the 3 PR for the testers.

@jma jma merged commit 73b977f into rero:dev Feb 11, 2021
@iGormilhit iGormilhit modified the milestones: v1.1.0, v1.0.1 Mar 1, 2021
@jma jma deleted the maj-new-editor branch January 13, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment