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

Merge Bug Fixes #1047

Closed
wants to merge 13 commits into from
Closed

Merge Bug Fixes #1047

wants to merge 13 commits into from

Conversation

vmw
Copy link
Contributor

@vmw vmw commented Sep 9, 2019

This branch applies the following bug fixes;

fixes: #997, #989, #418, #1002

It also fixes exporting issues, allows the root category to not be set as default, fixes prices in stoc history, and fixes a rounding issues in parameter searching.

RezaMohammadi99 and others added 13 commits June 26, 2019 21:55
This commit sets the default precision of part parameter
searching to femto to maintain fractional values.
This commit adds the part price information to the stock history.
The average price of a part doesn't change when stock is depleted
This commit sets the default category to null upon part creation.
This is in effort to make the user pick the correct category.
This commit fixes the exporter adding accented characters to the file.
This commit keeps the project remark connected to
the newly selected items in a project report.
* populate all form-items with values from selected parameter-record
 `onParameterSelect(...)`

* normalize order of properties (list unit name after id)

Signed-off-by: Florian Kerle <[email protected]>
This commit closes issue partkeepr#418.
Double clicking on a part-item in a 'Storage Location' opens
the part details window.
Do not rely on deprecated the URL.createObjectURL for MediaStream, other than as a fallback for old browsers.

Firefox (62+) have deprecated it, Safari have deprecated it, and Chrome have started the process to deprecate this as well.
I should note that I have not built this code myself and tested it, but the issue partkeepr#1002 is still present in the demo site and this code here is simply adopted from the developer site of Mozilla.

Sources:
https://bugs.webkit.org/show_bug.cgi?id=167518
https://bugs.chromium.org/p/chromium/issues/detail?id=800767#c28
https://www.fxsitecompat.com/en-CA/docs/2018/url-createobjecturl-no-longer-accepts-mediastream-as-argument/
https://developer.mozilla.org/sv-SE/docs/Web/API/HTMLMediaElement/srcObject#Supporting_fallback_to_the_src_property
@vmw
Copy link
Contributor Author

vmw commented Sep 16, 2019

Viewing the CI build, I think I know what happened; some files were incorrectly committed. I'm closing this request and pushing a new one.

@vmw vmw closed this Sep 16, 2019
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.

Duplicate button "Expand all Groups" should be "Collapse all Groups"
7 participants