-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Closed
Labels
type:docsOnly change in the docs or READMEOnly change in the docs or README
Description
New Issue Checklist
- I am not disclosing a vulnerability.I am not just asking a question.I have searched through existing issues.I can reproduce the issue with the latest version of Parse Server.To pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Issue Description
The Parse Server options SchemaOptions
which were added in #7418 were not properly added to the definition file.
The SchemaOptions
are defined in /src/Options/Definitions.js
, but when re-building the options with npm run definitions
, the options are removed.
Steps to reproduce
- Run
npm run definitions
Actual Outcome
Schema options are removed from /src/Options/Definitions.js
.
Expected Outcome
Schema options should be properly re-built.
Environment
Server
- Parse Server version:
5.0.0-alpha.24
Metadata
Metadata
Assignees
Labels
type:docsOnly change in the docs or READMEOnly change in the docs or README
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
parse-github-assistant commentedon Feb 12, 2022
Thanks for opening this issue!
mtrezza commentedon Feb 12, 2022
@sadortun @Moumouls could you take a look?
[-]Schema server option definition not properly added[/-][+]`SchemaOptions` definition not properly added[/+]dblythy commentedon Mar 19, 2022
What do you think about having
npm run definitions
ran / checked by the CI?mtrezza commentedon Mar 19, 2022
What do you mean by "checked by the CI"?
dblythy commentedon Mar 19, 2022
Well I’m assuming here that a contributor thought that they had to manually edit the Definitions.js file (even though it says “ Do not manually), instead of adding to index.js. I think it would be good to possibly have a CI check to make sure npm run definitions matches the committed Definitions.js file, so we can be sure no manual editing has been applied.
mtrezza commentedon Mar 19, 2022
Got it, I think we could simply generate the definitions file again and compare the result with what's been committed. Or do you have a different idea?
Moumouls commentedon Apr 3, 2022
The source of the problem is simply related to the fact that the definitions were added manually to the auto-generated file
mtrezza commentedon Apr 3, 2022
Thanks for fixing this, the discussion was about how to prevent that in the future. Generating the definitions file and comparing it to the current definitions seems an easy check for the CI. I opened #7918 so we close this issue with your fix.
mtrezza commentedon Apr 3, 2022
Closed via #7917