chore: regular update

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.27%. Comparing base (0cb009e) to head (a2c80de).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #103   +/-   ##
=======================================
  Coverage   85.27%   85.27%
=======================================
  Files          14       14
  Lines         523      523
=======================================
  Hits          446      446
  Misses         77       77

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Ascensia meter listing (UPLOAD-902, UPLOAD-858)

Addresses UPLOAD-902 and UPLOAD-858.

@krystophv This one just needs a quick review as it now also adds the Contour Next GEN ([UPLOAD-858]).


TODO - CNN Orthogonality regularatization

Can We Gain More from Orthogonality Regularizations in Training Deep CNNs?, 2018

added soft orthogonality constraint


Unstyled (or misplaced) UI elements on the market page

Firefox/Win7
2.0.151125
There are probably unintended (or badly styled) UI elements as part of the text input fields.
Happens only in FF/Win 7, in Chrome those elements do not show up at all.

That's how FF handles number fields. This falls under #474 in IMO. I really hope you agree. @neura-sx

It's due to a CSS hack to remove the up/down arrows that apparently FF ignores, I can have a look to see if there's also a FF version.

I agree with wmbutler. So it's up to svk to close this issue if there is no easy solution for the CSS hack.


Fix #34 - Use target type through proxy-target

This is a WIP as it's still unclear how many changes I need and where or why but I am working on it ...

Helping myself to deal with required changes as investigation goes on:

the Bound is needed to wrap an entry (pair) that points at something else that's not directly a function (the pair matters, not the function, as the type can be "function" but the value is not necessarily a function, rather an id)
currently everything works up to the array, as I've changed the default entry to a pair which doesn't play well anymore with Array.isArray drilling of the specs
there is a passive proxy that just invokes traps and return results and an active one that must fulfill the standard ... this one should deal with objects or functions, and it does it already, but it's incapable of dealing with arrays now and this is the main bummer

Things to do next:

use wrap from proxy-target to return an entry instead
still use Bound helper to return a function that contains an hybrid entry
when wrap returns an Array, be sure the values are normalized
that is: check typeof function or object and object !== null and isArray(object) all over anywhere


Fix Trade History command, swap Sold and Bought.

Text "Sold" and "Bought" was in wrong order. So I swap two text in right order.

That would be a wrong solution. Something else is wrong. My trade history is in the correct order

Okay so in one of my latest commit I did that: 237523cbd9429de15edc17b3aff5caf352c25120
Where I swapped bought and sell.
The issue was that the french translation is in the wrong order, jokes on me.

@MasaiasuOse you might be not up to date

@all-contributors please add @hieu0nguyen for bug


Never import "NA" and "NS" values; treat them as empty cells (seen in generated "NA" R entities)

Example of input: https://docs.google.com/spreadsheets/d/13eVorFf7J9R8YzO7TmJRVLzIIwRJS737r7eFbH1boyE/edit#gid=2056508047&range=CS7
Example of problematic output: T "Sentences of William Arnold and Stephen of Saint-Thibéry":

Desired output: do nothing, do not generate any R with label "NA" here.

Done.


Fixing up eslint warnings, and cleanup

Motivation
Remove a few standing warnings
Changes

Fixes existing warnings
Turns on/up eslint rules we mostly already follow

Testing Instructions

hi i would like to work on this issue?

@sumitshinde-84 This is a pull request, the change is already ready

Ok sir , let me know if you have any task for me.


[BUG]The pre-trained model of cellpose3 cannot be downloaded

The pre-trained file of https://www.cellpose.org/models/cyto3torch_0 does not exist.
when I use the code model = models.Cellpose(gpu=use_GPU, model_type='cyto3'), an error is reported:
2024-07-25 02:45:16,707 [INFO] Downloading: "https://www.cellpose.org/models/cyto3torch_0" to /h3cstore_nt/mm/cellpose/models/cyto3torch_0
Traceback (most recent call last):
File "/h3cstore_nt/mm/cellpose/BBBC/123.py", line 62, in
model = models.Cellpose(gpu=True, model_type="cyto3")
File "/opt/conda/lib/python3.8/site-packages/cellpose/models.py", line 105, in init
self.pretrained_model = [model_path(model_type, j, torch) for j in model_range]
File "/opt/conda/lib/python3.8/site-packages/cellpose/models.py", line 105, in
self.pretrained_model = [model_path(model_type, j, torch) for j in model_range]
File "/opt/conda/lib/python3.8/site-packages/cellpose/models.py", line 28, in model_path
return cache_model_path(basename)
File "/opt/conda/lib/python3.8/site-packages/cellpose/models.py", line 41, in cache_model_path
utils.download_url_to_file(url, cached_file, progress=True)
File "/opt/conda/lib/python3.8/site-packages/cellpose/utils.py", line 69, in download_url_to_file
u = urlopen(url)
File "/opt/conda/lib/python3.8/urllib/request.py", line 222, in urlopen
return opener.open(url, data, timeout)
File "/opt/conda/lib/python3.8/urllib/request.py", line 531, in open
response = meth(req, response)
File "/opt/conda/lib/python3.8/urllib/request.py", line 640, in http_response
response = self.parent.error(
File "/opt/conda/lib/python3.8/urllib/request.py", line 569, in error
return self._call_chain(*args)
File "/opt/conda/lib/python3.8/urllib/request.py", line 502, in _call_chain
result = func(*args)
File "/opt/conda/lib/python3.8/urllib/request.py", line 649, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 500: INTERNAL SERVER ERROR

And I can't download it when I open the website online: https://www.cellpose.org/models/cyto3torch_0

I think you might be using an old version of cellpose? I'd upgrade to 3.9 or greater python and pip install cellpose --upgrade, please let us know if that doesn't fix the problem and I'll reopen the issue


Improve the contributor guide, esp. the create-extension examples

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/342/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/343/


Fix crash: firstObject is more secure because it returns nil

In some situations the framework crashes because the dispatch is asyncrhonous and it is accesing to the object with zero index, so to fix it it is better to use "firsObject" method because it returns nil and it does not crash.

Thanks! Nice catch. Approved.

Not sure why a crash is being generated when accessing a 0 index of an array. Above the 0 index access there are condition (if checks) that make sure the array has at least one object.. Can you give more details about the crash you are getting and why accessing via firstObject solves it?

Here you has the explication about firstObject or objectAtIndex: https://stackoverflow.com/questions/37045858/nsarray-difference-between-firstobject-and-objectatindex0
If you don't want to use firstObject you must do the check inside of the dispatch because it is asynchronous and it can crash.

Although for me, it is cleaner to use the "firstObject" method.
Here you can see the crash in Fabric of the app which I work.

Thanks you so much for your time!! :)

The link you posted is no longer working. When I did read it though I don't believe it was sufficient to explain the reasoning why firstObject should be used and how it ties in to the need to use it in the async call. Can you provide steps to reproduce? How exactly is the crash being generated?

I fixed the link if you want to read it. If I find the steps to reproduce it, I communicate you 👍

As far as the link you posted I understand why a direct access to the array element index 0 results in a crash when the array is empty. firstObject returns nil and prevents a crash. But that wouldn't solve the true problem which is reproducing how the array becomes empty in the async call but not right before the async call in the if check. Definitely let me know if you can reproduce the issue.

I believe the issue lies in that the dismissNotification method is not thread safe. If multiple messages are displayed and then auto dismissed the timing of the dismiss notifications may possibly lead to a crash. I haven't completely verified this yet but expect an update soon.

Can you please try the develop branch and tell me if you still get the crash or can reproduce?

ok, I am going to use develop branch and if I can reproduce the crash I will tell you :)

also test the regular master branch.. there were some slight modifications there as well that "may" have solved the crash.


Use octokit instead of simple-get

For some reason, simple-get would just timeout when using gh-release on some orgs.  I thought it was an issue with the device token permissions, but its not.  The GitHub API is weird, and finicky, and IME using octokit, despite its constant churn, is probably the best approach here.  It also generally improves errors that get returned, though most of our error handling I left the same.
I've tested the following scenarios:

Normal working scenario
Missing tag
Bad credentials
Asset uploads

Everything still seems to work.

I would basically count this as a bug fix since simple-get didn't quite work out.

seems reasonable but I'd like to do some extra manual testing on it since it's a pretty non-trivial rewrite of core logic (replacing the thing that creates the github release itself).
if any other maintainers are able to get to it before me and feel like this is solid and non-breaking, feel free to merge & release

I'm going to cut a beta release to test this out in the wild some more.

Ok npm i gh-release@beta will install the beta.  Going to use it out in the wild for a bit

@bcomnes any news on this one? sorry i'm spread pretty thin and not really maintaining anymore. if this is working well for you i'm fine approving and you can do a major release

@bcomnes any news on this one? sorry i'm spread pretty thin and not really maintaining anymore. if this is working well for you i'm fine approving and you can do a major release

@bcomnes any news on this one? sorry i'm spread pretty thin and not really maintaining anymore. if this is working well for you i'm fine approving and you can do a major release

Ok yeah no problem.  Hope everything is as well as it can be.
It fixed my issue that I was having with the current approach, and has worked in a few other places.  I'm happy to monitor a release on this, and it should fix weird edge cases that I cannot determine the cause for with the GH api.

@bcomnes any news on this one? sorry i'm spread pretty thin and not really maintaining anymore. if this is working well for you i'm fine approving and you can do a major release

Ok yeah no problem.  Hope everything is as well as it can be.
It fixed my issue that I was having with the current approach, and has worked in a few other places.  I'm happy to monitor a release on this, and it should fix weird edge cases that I cannot determine the cause for with the GH api.

Ok doing a major on this.

Ok doing a major on this.


fix: for DA reduce validation on event notifications

Description
Reduce the validation on the enable_event_notifications in the DA.
Permit an EN CRN to passed and enable_event_notification to be false.
This is to support the stacks use case, where an existing SM and EN are passed, allowing the enable_event_notifications flag to be true or false to control the creation (or not) of additional resources.
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
DA only validation updates to better support stacks use case.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

Adding Todd and Igor as potential reviewers. I think this will work for a stacks scenario now. Passing false should skip over all the resources in the DA and the main module relating to EN and SM/EN integration.

:tada: This issue has been resolved in version 1.17.6 :tada:
The release is available on:

GitHub release
v1.17.6

Your semantic-release bot :package::rocket:


Component board

Create a visual component board to help identify that pf-next styling and serve as a guide for development.
Refine and organize the component board to be leveraged as a reference for the broader team. Also includes incorporating accessibility work.

Board is updated
Organize the files and folders
Review with visual design team
Share the link to the PF Swarm team

@LHinson @kybaker - here is the updated component board! Feel free to close :) https://drive.google.com/drive/folders/1kOzgBU64TDkMNoi_1YX9NQ4kyu1OmRX6?usp=sharing

Thanks @jennyhaines!


chore: Drop Python 3.8 support. Add Python 3.13 support. Update CI/dev dependencies.

Might also be worth to move the mypy config from setup.cfg to pyproject.toml.

I don't exactly know what causes it, but for some reason installing flake8-use-pathlib with some of the other updated dependencies breaks a small number tests in 3.12+
I decided to get rid of flake8-use-pathlib and instead use flake8-bugbear for the regression test. It technically should be a more reliable test, since it's a more complex plugin, accessing more nodes.

@sondrelg The CodeCov issue is something you'd probably need to sort out yourself, by adding the CODECOV_TOKEN to the repository secrets and passing it to the codecov action, they recently started requiring a token, even for open source projects.

Actually nevermind, looks like you are already supplying it, but it's not being used for PRs
Maybe fail_ci_if_error should be false for PRs.


Not all Playlists show up in Frontend

What version of Music Assistant has the issue?
2.3.2
What version of the Home Assistant Integration have you got installed?
2024.11.4
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Discord Discussion with Traveling_Gav about the issue here
When trying to add a track to a playlist from the MA Frontend, it opens the dialog of playlists, but it only shows a subset of my playlists + options to add new playlist to MA or Spotify.
A lot of my playlists from Spotify are prefixed with - and <space> making names like
- rock
- top40
- chill
- rap

All the playlists which follow this format are shown correctly, but no other playlist  is shown from any music providers. Adding a new playlist in Spotify with this prefix makes it show up, and same if I create a MA playlist with this prefix it show up, but all other playlists do not show up

Adding a new playlist works, but it does not show up unless I give it a name with the same prefix
The "Playlist" view in the frontend shows all the playlists correctly as expected

How to reproduce
Create playlist named like - <something> and then try to add a track to a playlist in the frontend, and see what is being shown

Search for Track
Click on ...
Click Add to Playlist...

Then only a subset of playlists are shown
Music Providers
Spotify
Filesystem (Remote)
Player Providers
N/A
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.11.1
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Linux

So I did some more digging into this, and think I found where the issue might be.
In AddToPlaylistDialog.vue where it executes getLibraryPlaylists, this for me, returns a array of exactly 500 elements. But I have a lot more than 500 playlists (closer to 1000, because each album on disk has a playlist).
Most of those playlists on disk show up in MA as editable: false, which make sense, and that's why they are not shown in the frontend.
I tested by simply removing all the playlists on disk, and now everything works as expected in the frontend, and it shows my playlists like I expect.
Not sure if there is some limit from the API making it return only 500 elements, or if some bug happens here?


Make core available as standalone consumable modules

Opening this early and unpolished to illustrate what it could look like and to discuss requirements. I tried to stay as close as possible to your original intent/style, but can totally understand if this is too much of a change for you. (I can always just use this forked code myself, as I don’t imagine there will be much changes in the future.)
Changes:

Changes to ES6 modules loadable as-is by modern browsers; which means no IE support out-of-the-box. The reason for this is that some form of exporting was necessary to consume these modules elsewhere. (CommonJS exports would always require some extra step/tooling to support browsers.)
Adds some more JSDoc to allow the use of the TypeScript compiler to provide some type-checking and IDE features.
Replaced minified version of notie with original ES6 version.
Initial refactoring to move browser specific code out into its own module.

Considerations:

Transpile to bundle that supports IE? Requires build-time tooling.
Perhaps rename files as .mjs to signal they are ES6 modules?
Perhaps change to use ES6 classes, which mostly means being able to have the TS compiler infer more with less docs needed (I think).
Need to figure out the best way to provide input data, as the HTML5 File API does not exist natively in NodeJS (nor does any polyfill package seem to provide all the relied upon APIs).

Closes #41

Hi, Sorry for the late reply. Have been a bit busy.
Although I am very glad you put so much effort into this, I think these changes are too big for me personally. Especially since we are talking about a small feature. If I were to create an electron build of the system, I think I would be okay with this. However I believe the system is easier and just as safe to use in the browser browser.
You are of course free to fork this project and change all the code as needed.
The only changes that I can foresee (except bugfixes and added support for banks which are easily integrated into your fork) are a merge with the ABN AMRO version of YNABGoingDutch #25  and an integration into the YNAB api #26 . However these features are not guaranteed to be implemented ever.
I can always link to your repo in the readme.md if you'd like. Thanks again for the effort you put into this!

Makes total sense and no worries, I need this so there’s nothing lost here.

The only changes that I can foresee (except bugfixes and added support for banks which are easily integrated into your fork) are a merge with the ABN AMRO version of YNABGoingDutch #25 and an integration into the YNAB api #26 . However these features are not guaranteed to be implemented ever.

I’m tracking the API one, but also don’t foresee it being a real problem to merge back in if it ever gets done.

I can always link to your repo in the readme.md if you'd like.

Yeah that would be nice, I’m sure there will be some other dev at some point that wants this 👍

Thanks again for the effort you put into this!

Thank you for the original work!


Fix HTTP request sending in auth code

@JeffreyRichter, _az_credential now needs the http_client_fn to send requests, this PR adds that.
@vhvb1989, there was a bug in our CURL code - when we used AZ_RETURN_IF_CURL_FAILED against az_result (i.e. instead of AZ_RETURN_IF_FAILED). I've made naming changes as well, and also marked all the local non-AZ_INLINE functions as static, so these internal function are never visible to linker outside of that C file (but that is not critical for the bugfix).
[ATTENTION: Make sure you see the changes to the az_curl.c (at the very end of this PR) - Github hides the change claiming it is "large" to show by default, so you need to click to expand]
For some reason, I am getting HTTP 400 (Bad Request), when I include Content-Type header for token request. But everything works fine if I don't add any header. But it works in Postman, so I think it is highly likely that there's a bug is in the code that handles HTTP headers - either CURL http client implementation, or in http_request.
With Content-Type commented out, this actually fixes auth token retrieval, although I would like to be able to include it for auth requests. Plus, if I am right, you won't be able to send any HTTP request anyways, if there's all-affecting http header bug in our code. If so, let's fix it tomorrow.
Also, do we use pointer to pointer to http_client client intentionally? We use it in all client options, plus http policy expects not the pointer to http client, but a pointer to pointer. Therefore, I also wrote credential_policy to stick tho that format. Is this intentional to allow switching http clients in run time? (i.e. you update keyvault client's http client after it was initialized, and it starts sending all further http requests via another client)

@JeffreyRichter , what is better ? : - to initialize credential with the client's http_client during the client's init() [this is what I do now], or to request another http_client parameter in the credential's init?


A new pull request

The Gremlin Code Removed and the fork is reverted to your last commit
FIXED: Changed the package name from exdgraph to ex_dgraphin README
Added a test case with "StarWars Sample Data"

But the same thing. The test fails with travis-ci.  And test success on my local machine.
  1) StarWarsSampleTest: failure on setup_all callback, test invalidated
     ** (EXIT from #PID<0.1316.0>) {:callback_mode, :ok}

Hey, I don't really know why your StarWarsSampleTest fails on Travis. I will have a bit more time towards the end of the week. Please note in general that everything I provided is very much under development and not just ready to use. So maybe there lies the issue.

@Fulnir I started to implement the DBConnection behaviour. A lot of things have changed already. Check it out!


Fix issue 178

#178
This fix is for the Issue from the link above. It was closed recently... However, I have been requested to fix this as a part of the GE team.

Taken forward in #207


Jmeter Test plan and execution

Great tool. Is it possible All the things we can do with JMeter can be done with JMeter-java-DSL?

Thank you very much.
In theory yes, but currently the api is limited to the most used cases and what users have requested.
If you are in need of something the api doesn't still support ,please create an issue requesting it and with some explanation why is needed, and we will be very glad and eager to implement it (usually is quite fast to implement support for new jmeter features).
You can also easily implement (if you have enough programmatic and jmeter knowledge) new missing features in the dsl. You can check the Contributing document for a quick intro.
Regards

Thanks


[DC-164] Updated eslint to warn on unused arguments as well as unused variables

Now, unused variables in function parameters will throw errors

One of the eslint changes to note: I updated the PR so that variables with "_" as the first character of the name will be exempted from this eslint rule. This is to allow for our use of external library functions without having to eslint:ignore all of the lines.
Some of the conditions to consider:

I added a bunch of "ignore-next-line"s to cases where I wasn't sure if it would break something. The "Ref" example happens a LOT and I don't know if removing that unused variable would affect anything?

This looks good. I found the PR with the unused variable. After some conversation I used _.unused. What are your thoughts on renaming all unused variables to _.unused? The thought on the other PR is that it becomes a bit more obvious that the variable is not used.

I think I want to keep the variable names what they are - here are some of my thoughts:

We might have multiple unused vars in function names
If we want to suddenly start using them, we can just delete the _
when debugging, having a name can help us identify what it's SUPPOSED to be for (esp for things like the _refs)


HomeBridge starting problem

I am in the process of installing HomeBridge on a Raspberry Pi B+ for the first time.  The only plugin I have installed is homebridge-smartthings.  When attempting to start HomeBridge I get the following warning & error messages:
Warning:
*** WARNING *** Please fix your application to use the native API of Avahi!
Error:
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::51826
    at Object.exports._errnoException (util.js:837:11)
    at exports._exceptionWithHostPort (util.js:860:20)
    at Server._listen2 (net.js:1231:14)
    at listen (net.js:1267:10)
    at Server.listen (net.js:1363:5)
    at EventedHTTPServer.listen (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/util/eventedhttp.js:58:19)
    at HAPServer.listen (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/HAPServer.js:155:20)
    at Bridge.Accessory.publish (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/Accessory.js:480:16)
    at Server._publish (/usr/local/lib/node_modules/homebridge/lib/server.js:92:16)
    at Server.<anonymous> (/usr/local/lib/node_modules/homebridge/lib/server.js:351:14)

Any advice on how to proceed?

Something is already running on port 51826. Looks like it may be trying to bind to your IPv6 interface too because it has two colons.

I think mine is connecting to IPV6 too, how can we stop that from happening?

Did you start it as root or as a normal user?
Because on my raspberry it only runs when started as root

@robbiet480, any advice on how to fix either of the problems you mentioned?  I'm fairly new to this and don't know where to start looking.
@thejacko12354, I just tried running sudo homebridge and am still getting the same error.

Open activity Monitor
Search for Homebridge
Kill Process
Hope that works for you

Isn't it quite similar to this issue https://github.com/nfarina/homebridge/issues/677 ?
I face the same problem, even if I don't have any other process already bound to port 51826, and even with IPv6 disabled in the OS 😣

I have this same error since a few days (all was running well before, and I have not made any changes on my rpi). However, changing the port does not solve the issue, so I do not think this is the root cause. In my case changing the port just give a dns name error now (I already posted that error in #725 )

Full reload of OS fixed if for me: ( same as #725 )

I'll give it a try this evening, as I can re-install my Rpi without any collateral damages :-) But it's a quite violent way to fix this problem !

@kemathy:
Agreed; but since I could rule-out a corrupt ad card, nor find the root-cause I figured this was the less evil option :-(

I ran into this problem as well, after reinstalling my Pi with a fresh homebridge configuration. I was able to install and set-up everything and it worked well. Then I followed the tutorial to set-up the autostart (https://timleland.com/setup-homebridge-to-start-on-bootup/) as as soon as I rebooted the Pi for the first time, I started to get this error.
I tried many things and then removed the bootup services. Since then, Homebridge worked well again. So I created the boot services again and paid a little more attention this time :))
Problem 1: wrong folders

stick to the instructions and gists here: https://gist.github.com/johannrichard/0ad0de1feb6adb9eb61a/
pay attention to the folders in which you installed homebridge: if you followed the easier node installation using apt.get as described in the main tutorial in this repo (https://github.com/nfarina/homebridge/wiki/Running-HomeBridge-on-a-Raspberry-Pi#running-homebridge-on-bootup), you need to change the homebridge.service contents as described in the gist (leave out "local" in ExecStart=/usr/local/bin/homebridge $HOMEBRIDGE_OPTS)

Problem 2: permissions

when launching homebridge as a service (not manually), it is necessary to pay more attention to the permissions
I had iKettle plugin for CMDswitch2 installed to the home folder of Pi user ... I moved it manually, changed the config.json and it works now

Problem 3: issues related to plugins

on top of all of this, I had an issue with one of the plugins as well
when multiple issues cross in your home setup, it is not always easy to find the solution (mainly in cases when a problem with plugin is able to influence homebridge so badly, that also other things quit working)

Hope this helps at least to some of you guys!


./src/index.js Cannot find file: 'Hello.js' does not match the corresponding name on disk: '.\src\hello.js'.

Bug report

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:
Node.js version:
Operating System:
Additional tools:

what can i do plz guide me to solve it...

import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import Hello from './Hello';
import * as serviceWorker from './serviceWorker';
ReactDOM.render(
<React.StrictMode>

</React.StrictMode>,
document.getElementById('root')
);
// If you want your app to work offline and load faster, you can change
// unregister() to register() below. Note this comes with some pitfalls.
// Learn more about service workers: https://bit.ly/CRA-PWA
serviceWorker.unregister();

import React from 'React';
import ReactDom from 'react-dom';
class extends Hello React.component{
render(){
return(

Welcome Coderworld!

)
}
}
export default Hello;

Have you created hello.js file in your React app src folder

You should keep the import hello from './Hello' reference and Hello.js filename consistent.

Please submit support requests and questions to StackOverflow using the tag [webpack]. StackOverflow is better suited for this kind of support. The issue tracker is for bug reports and feature discussions. See also our CONTRIBUTING guidelines.


Clean up WIP branches

There are a few WIP branches on this repo - it would be good to either delete them (if no longer wanted) or finish them up.
These two seem redundant after #28:
https://github.com/Malax/libcnb.rs/compare/build_context_new?expand=1
https://github.com/Malax/libcnb.rs/compare/malax/typed-layer-metadata-proto?expand=1
This one may still be relevant:
https://github.com/Malax/libcnb.rs/compare/buildpack_toml_serialize?expand=1

I removed mine.

@hone Could you see if these branches of yours are still needed? :-)
https://github.com/Malax/libcnb.rs/compare/build_context_new?expand=1
https://github.com/Malax/libcnb.rs/compare/buildpack_toml_serialize?expand=1

yeah, I think the buildpack toml serialize is the only one I want to keep. Manuel kind of broke the world with 0.2.0. :|


Spacing issue in menus and windows.

Force recheck window

On right clicking an errored torrent.

This is present in both mica and no-mica themes. Also, this is the state of mice for everyone.

fixed

The popup window sizing is fixed, but the menu sizing issue still remains. It happens when I click on an errored torrent.

i tried to find a way to fix it, but the only thing that worked was to change the color of the selection so that it wasn't very conspicuous


Can I use mitmproxy as a upstream backend?

In my job, mitmproxy speed is bottleneck.
I try to make load balancer with nginx server.
I set my mitmproxy processes as upstream back end
Below example setting is my nginx setting
http{
    upstream mymitmproxy {
        server mitmproxy.example1.com;
        server mitmproxy.example2.com;
        server mitmproxy.example3.com;
    }

    server {
        listen 8080;

        location / {
            proxy_pass http://mymitmproxy;
        }
    }
}

I check the traffic is transffered to mitm proxies but there is an error occur.
<< Invalid HTTP request form (expected: authority or absolute, got: relative)
HTTP protocol error in client request: Invalid HTTP request form (expected: authority or absolute, got: relative)
Somebody help me :)
Thanks.

Sorry, It was my configuration problem.
It works new version (3.03)


Support angular-universal cache priming

This captures the HTTP requestes issued during server-side rendering, and packages them with the page so the client-side code doesn't re-issue the same requests.
Currently the implementation in angular-universal itself is incomplete (doesn't support current Angular 2 http APIs), so we need to wait for that.

Is it different from Microsoft.AspNetCore.AngularServices.PrimeCacheHelper.PrimeCache or Microsoft.AspNetCore.AngularServices.PrimeCacheHelper.PrimeCache is incomplete?

It's not completed on the Universal side @andreymir. We have a spec out for it, but it hasn't been fully put together or incorporated just yet.

While this may be nice to do one day, there's no ETA for the feature on the Universal side, and very few people have expressed interest in having the feature anyway.
I'll close this. We can always reconsider including it once it does ship in Universal and if people are keen to have some integration with it.


Migrate static_text.test.ts to North Star

Instructions
See https://github.com/civiform/civiform/issues/8791

@ssandbekkhaug - the static_text.test.ts  test file already has test cases for north star: https://github.com/civiform/civiform/blob/3c04e44626b0b59e266c2da5bc8e2efc432e8d4d/browser-test/src/applicant/questions/static_text.test.ts#L61.   Are you asking to split the North Star test cases into a separate file?

Yes. Please see the full instructions linked in #8791


Plugin incompatible with Windows - WAMP, XAMP, MAMP 4 Win,etc

Does work on AMPPS, which appears to use Unix filepaths
reported issues by several users.
Filepaths, in particular, seem to be the issue.
Plugin needs to detect Windows servers and adjust filepath building functions accordingly.

attaching example export log from a Windows MAMP export, showing the malformed half Windows/Unix paths, also, the log file was generated with a .bin extension, all the more reason to store it in the database, as per other issue which will do as such.
WP-STATIC-EXPORT-LOG.bin.txt

including example export log from Windows MAMP server installation:
[06-Jul-2018 10:06:06 UTC] PHP Warning:  fopen(C:\MAMP\htdocs\wp/wp-content/uploads/WP-STATIC-EXPORT-LOG): failed to open stream: No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 695
[06-Jul-2018 10:06:06 UTC] PHP Warning:  stream_copy_to_stream() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 698
[06-Jul-2018 10:06:06 UTC] PHP Warning:  rewind() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 700
[06-Jul-2018 10:06:06 UTC] PHP Warning:  stream_copy_to_stream() expects parameter 2 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 701
[06-Jul-2018 10:06:06 UTC] PHP Warning:  fclose() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 702
[06-Jul-2018 10:06:07 UTC] PHP Warning:  fopen(C:\MAMP\htdocs\wp/wp-content/uploads/WP-STATIC-EXPORT-LOG): failed to open stream: No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 695
[06-Jul-2018 10:06:07 UTC] PHP Warning:  stream_copy_to_stream() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 698
[06-Jul-2018 10:06:07 UTC] PHP Warning:  rewind() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 700
[06-Jul-2018 10:06:07 UTC] PHP Warning:  stream_copy_to_stream() expects parameter 2 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 701
[06-Jul-2018 10:06:07 UTC] PHP Warning:  fclose() expects parameter 1 to be resource, boolean given in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 702
[06-Jul-2018 10:06:41 UTC] PHP Warning:  symlink(): Cannot create symlink, error code(1314) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 620
[06-Jul-2018 10:06:41 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-content,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/contents): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 577
[06-Jul-2018 10:06:41 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/contents///themes,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/contents/ui): The system cannot find the path specified. (code: 3) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 581
[06-Jul-2018 10:06:41 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/contents/ui/twentyseventeen,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/contents/ui/theme): The system cannot find the path specified. (code: 3) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 582
[06-Jul-2018 10:06:41 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-includes,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/inc): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 586
[06-Jul-2018 10:06:41 UTC] PHP Warning:  scandir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-json/,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-json/): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 10:06:41 UTC] PHP Warning:  scandir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-json/): failed to open dir: No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 10:06:41 UTC] PHP Warning:  scandir(): (errno 2): No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 10:06:41 UTC] PHP Warning:  array_diff(): Argument #1 is not an array in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 10:06:41 UTC] PHP Warning:  Invalid argument supplied for foreach() in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 19
[06-Jul-2018 10:06:41 UTC] PHP Warning:  rmdir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530871568-grace/wp-json/): No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 23
[06-Jul-2018 13:07:30 UTC] PHP Warning:  PHP Startup: Unable to load dynamic library 'php_mysql.dll' (tried: C:\MAMP\bin\php\php7.2.1\ext\php_mysql.dll (The specified module could not be found.
), C:\MAMP\bin\php\php7.2.1\ext\php_php_mysql.dll.dll (The specified module could not be found.
)) in Unknown on line 0
[06-Jul-2018 13:07:35 UTC] PHP Warning:  PHP Startup: Unable to load dynamic library 'php_mysql.dll' (tried: C:\MAMP\bin\php\php7.2.1\ext\php_mysql.dll (The specified module could not be found.
), C:\MAMP\bin\php\php7.2.1\ext\php_php_mysql.dll.dll (The specified module could not be found.
)) in Unknown on line 0
[06-Jul-2018 13:13:19 UTC] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in C:\MAMP\htdocs\wp\wp-admin\includes\class-wp-filesystem-direct.php on line 76
[06-Jul-2018 13:16:57 UTC] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in C:\MAMP\htdocs\wp\wp-includes\pluggable-deprecated.php on line 18
[06-Jul-2018 13:17:05 UTC] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in C:\MAMP\htdocs\wp\wp-includes\class-wp-roles.php on line 26
[06-Jul-2018 13:17:06 UTC] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in C:\MAMP\htdocs\wp\wp-includes\class-wp-meta-query.php on line 22
[06-Jul-2018 13:20:45 UTC] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in C:\MAMP\htdocs\wp\wp-admin\includes\import.php on line 17
[06-Jul-2018 13:27:48 UTC] PHP Warning:  symlink(): Cannot create symlink, error code(1314) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 620
[06-Jul-2018 13:27:49 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-content,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/contents): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 577
[06-Jul-2018 13:27:49 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/contents///themes,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/contents/ui): The system cannot find the path specified. (code: 3) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 581
[06-Jul-2018 13:27:49 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/contents/ui/twentyseventeen,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/contents/ui/theme): The system cannot find the path specified. (code: 3) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 582
[06-Jul-2018 13:27:49 UTC] PHP Warning:  rename(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-includes,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/inc): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput.php on line 586
[06-Jul-2018 13:27:49 UTC] PHP Warning:  scandir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-json/,C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-json/): The system cannot find the file specified. (code: 2) in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 13:27:49 UTC] PHP Warning:  scandir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-json/): failed to open dir: No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 13:27:49 UTC] PHP Warning:  scandir(): (errno 2): No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 13:27:49 UTC] PHP Warning:  array_diff(): Argument #1 is not an array in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 17
[06-Jul-2018 13:27:49 UTC] PHP Warning:  Invalid argument supplied for foreach() in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 19
[06-Jul-2018 13:27:49 UTC] PHP Warning:  rmdir(C:\MAMP\htdocs\wp/wp-content/uploads/wp-static-html-output-1-1530883630-grace/wp-json/): No such file or directory in C:\MAMP\htdocs\wp\wp-content\plugins\static-html-output-plugin\library\StaticHtmlOutput\FilesHelper.php on line 23

Use local by flywheel to build your wordpress site locally on windows or mac. It's work perfectly.

migrated to https://wp2static.talkyard.net/-9/plugin-incompatible-with-certain-windows-servers


Lua Network API

Exported the cNetwork API to Lua, so that plugins can use it.

I'll rebase this onto current master to make it auto-mergable again.

I think this might be ready for a merge

:+1:


fix build production only!!

eval( ) problem on production!! csp unsafe

StencilJS is complaining about it also:
[ WARN  ]  Bundling Warning
           Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification

Is there a real reason, why you want to use the minified version? If you use a production ready toolchain, your toolchain will include and minify this in your build.

no the only reason is that in 3.4.0 the NON min version is a DEV version with eval( ....
and if you are on a server with CSP that does not allow eval unsafe.eval all is broken!!
i think you shoud release in dist/ only PRODUCTION env code, min and non min version

https://github.com/krisk/Fuse/blob/master/dist/fuse.js#L114

So, your proposed fix is a hotfix and not a correct fix.
You may want to look at #280, where an similar issue is described.

StencilJS is complaining about 3.4 (it was not in 3.3). I don't know if it
imports dev or production, and I don't really care. For me, just stopped
working.
[ WARN  ]  Bundling Warning
Use of eval is strongly discouraged, as it poses security risks
and may cause issues with minification
On Thu, Feb 7, 2019 at 1:15 PM Johann Wagner<EMAIL_ADDRESS>wrote:

So, your proposed fix is a hotfix and not a correct fix.
You may want to look at #280 https://github.com/krisk/Fuse/issues/280,
where an similar issue is described.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/krisk/Fuse/pull/283#issuecomment-461463078, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAXJyzGB9LiM-A7J_GEdEFBXOnCnNs4iks5vLEMMgaJpZM4anKGY
.

i think you are missing the point.
dist/fuse.js is built in DEV MOE and is exported from package JSON -> main: "dist/fuse.js"
dist/fuse.min.js is bult in PROD mode but is NOT exported anyware.
ok?
i'm pushing a FULL fix to webpack.config.js
give me 5 minutes

I totally got your point.
I just wanted to point out, that it is not the solution to use the minified version, otherwise you will kill tree shaking for other users. You have to fix the non minifed build, this also should be production ready, because it is in the dist folder.

@johannwagner see now. thanks

Taking a look at this now. Thanks for flagging. Broken behavior right now.

thank you ! all is working now!


TabIcon failed to respond to use click in nested navigator

Current Behavior

In my app, I have a login screen and a screen with tabbars at the bottom of the screen. I used stacknavigator to handle login screen and the screen with tabbars. If the use logged in successfully, I switched to tabnavigator. I used redux in my app. When I clicked the icon on the tab bar, I can see the state of my app changed. But the state of the icon does not change. Only the initial icon is active no matter which icon on the tab bar I clicked.
Expected Behavior

If a tabnavigator is nested in a stacknavigator, I should be able to choose different tabs.
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
0.44.0

node
v8.1.0

npm or yarn
5.0.3

I just noticed that the app worked as expected if I enabled swipe. Maybe there's some sort of problem in my code?

Thanks for the issue. Can you provide more information about your navigation stack and/or code snippets that you believe are causing the issue?

Thanks for your attention. I added 'use strict'; to the top of my code and the app is working as expected now.


ERROR: Gradle DSL method not found: 'implementation()'

Followed the steps in the Install section of README (modifications of build.gradle are available here), but failed to build the project:
$ ./gradlew assembleDebug
...
FAILURE: Build failed with an exception.

* Where:
Build file '/home/debug/codes/textile/android-textile/build.gradle' line: 12

* What went wrong:
A problem occurred evaluating root project 'android-textile'.
> Could not find method implementation() for arguments [io.textile:textile:1.1.0] on object of type org.gradle.api.internal.artifacts.ds$.dependencies.DefaultDependencyHandler.
...

Changed 1.1.0 to 2.0.5 will get the same error.

Environment

android-textile: 2.0.5 (commit 6771506)
Android Studio: 3.4.2
Gradle: 5.1.1 (indicated in gradle/wrapper/gradle-wrapper.properties)
Gradle plugin: 3.4.1 (indicated in manifest.gradle)

I haven't been able to reproduce this using the same environment as you. I did run into two other issues, I'm not sure if they would impact what you're experiencing:

I had to update to minSdkVersion 24 in app/build.gradle to be compatible with Textile 2.0.5
I had to add the following compile options to app/build.gradle in the android section:

compileOptions {
    sourceCompatibility 1.8
    targetCompatibility 1.8
}

Can you share your build.gradle files?

Thanks for your information and and my build.gradle is available here.
My app/build.gradle is the one in current master branch including:
    defaultConfig {
        ...
        minSdkVersion minSdk    // minSdk is 24 in manifest.gradle
        ...
    }
    ...
    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }

Side note... I just updated to Android Studio 3.4.2, default new project configuration, and no problems getting Textile integrated aside from the two tasks I mentioned in my previous comment.
I see a couple of potential problems with the config files you sent.
The most problematic is that you're using a project() dependency for Textile:
implementation project('io.textile:textile:2.0.5')

That is wrong. You should use:
implementation 'io.textile:textile:2.0.5'

Possibly less problematic is that you're defining the Textile dependency in your top level build.gradle... App dependencies should go in your app module's build.gradle.
I'm a little confused about your link to android-textile's build.gradle when referencing your app/build.gradle. Your app should have it's own app/build.gradle that included the dependency on Textile.

@asutula Thanks for the helpful tips. The root cause is that I added the Textile dependency in <project-root>/build.gradle instead of <project-root>/app/build.gradle.
I can build the app successfully by applying the modifications below:
diff --git a/app/build.gradle b/app/build.gradle
index 881f1d9..2ade019 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -73,4 +73,6 @@ dependencies {

     // Espresso dependencies (maybe needed in the future)
     // androidTestImplementation "androidx.test.espresso:espresso-core:$androidxTestEspressoVersion"
+
+    implementation 'io.textile:textile:2.0.5'
 }
diff --git a/build.gradle b/build.gradle
index 6f27426..5c6c134 100644
--- a/build.gradle
+++ b/build.gradle
@@ -16,9 +16,8 @@ allprojects {
     repositories {
         google()
         jcenter()
-        maven {
-            url 'https://jitpack.io'
-        }
+        maven { url "https://dl.bintray.com/textile/maven" }
+        maven { url 'https://jitpack.io' }
     }
 }

Glad to hear it :)


node_create: autofill & datepicker only work for the first form

Describe the bug
The autofill & datepicker plugins only work for the first form in the form-fields
To Reproduce
Steps to reproduce the behavior:

Create or edit node
Switch input type to "date"
Click on first text-field, datepicker is shown
Add a second logic form
Click first-text-field, datepicker is not shown
Same behavior for autofill for second text-field

Expected behavior
Datepicker/Autofill should work for multiple forms
Additional context
The jQuery listener is attached when document.ready is triggered. The jquery listener must be attached if a new form is added using dynamic formsets plugin. The plugin supports calling a function after a new form is added, integrate a function to attach the listener.

Solved in last commit.


:bug: CI should pass when on codecov upload fail.

Summary
Codecov becomes because github api outage sometimes. Change to pass CI job when the codecov upload is failed.
Related issue(s)
https://github.com/codecov/codecov-action/issues/926

/assign @qiujian16

/approve
/lgtm


cowboy_http_rest:process_post can accept location

It would be handy if process_post will accept Location value in the same way as put_resource.

What do you mean? If post_is_create/2 returns true then create_path/2 allows you to set a location.


ci: fix branch ISO has incorrect file name

Fix CI upload wrong branch ISOs: https://github.com/harvester/harvester/actions/runs/9283132460/job/25544321119

@mergifyio backport v1.3


docker-compose error when trying to run backend.

When I run "sudo docker-compose up -d" in the root directory I find this error:
Traceback (most recent call last):
File "urllib3/connectionpool.py", line 677, in urlopen
File "urllib3/connectionpool.py", line 392, in _make_request
File "http/client.py", line 1252, in request
File "http/client.py", line 1298, in _send_request
File "http/client.py", line 1247, in endheaders
File "http/client.py", line 1026, in _send_output
File "http/client.py", line 966, in send
File "docker/transport/unixconn.py", line 43, in connect
FileNotFoundError: [Errno 2] No such file or directory
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "requests/adapters.py", line 449, in send
File "urllib3/connectionpool.py", line 727, in urlopen
File "urllib3/util/retry.py", line 403, in increment
File "urllib3/packages/six.py", line 734, in reraise
File "urllib3/connectionpool.py", line 677, in urlopen
File "urllib3/connectionpool.py", line 392, in _make_request
File "http/client.py", line 1252, in request
File "http/client.py", line 1298, in _send_request
File "http/client.py", line 1247, in endheaders
File "http/client.py", line 1026, in _send_output
File "http/client.py", line 966, in send
File "docker/transport/unixconn.py", line 43, in connect
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "docker/api/client.py", line 205, in _retrieve_server_version
File "docker/api/daemon.py", line 181, in version
File "docker/utils/decorators.py", line 46, in inner
File "docker/api/client.py", line 228, in _get
File "requests/sessions.py", line 543, in get
File "requests/sessions.py", line 530, in request
File "requests/sessions.py", line 643, in send
File "requests/adapters.py", line 498, in send
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "bin/docker-compose", line 3, in
File "compose/cli/main.py", line 67, in main
File "compose/cli/main.py", line 123, in perform_command
File "compose/cli/command.py", line 69, in project_from_options
File "compose/cli/command.py", line 132, in get_project
File "compose/cli/docker_client.py", line 43, in get_client
File "compose/cli/docker_client.py", line 170, in docker_client
File "docker/api/client.py", line 188, in init
File "docker/api/client.py", line 213, in _retrieve_server_version
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
I've tried running it in the /backend folder but docker-compose does not find the env file as I assume is the expected behavior:
ERROR: Missing mandatory value for "networks" option interpolating ['${TRAEFIK_PUBLIC_NETWORK?Variable not
set}', 'default'] in service "proxy": Variable not set
I'm probably doing something wrong... Any thoughts?

Docker version 19.03.13, build 4484c46d9d
docker-compose version 1.27.4, build 40524192

I think I found the issue. The problem is due to something with my linux environment running on windows (WSL). I found a thread on it here:
https://github.com/docker/for-linux/issues/535
I will switch to my mac as I can not find a quick solution for solving this on my PC :)

Thanks for reporting back and closing the issue :+1:

The issue is not related to this project, it's docker-compose's problem..
The only thing you have to do is to add your user to the docker group:
sudo usermod -aG docker $USER
and then reboot your PC
after login do NOT use sudo to run docker-compose:
docker-compose up -d

I'm facing this issue as well on MacOS.
Here's my docker & docker-compose version, and the docker-compose.yml file. I'm not sure what the issue is, haven't been able to find related issues.
docker-compose version 1.29.2, build 5becea4c
Docker version 20.10.7, build f0df350

version: "3.3"
services:

  proxy:
    image: traefik:v2.2
    networks:
      - ${TRAEFIK_PUBLIC_NETWORK?Variable not set}
      - default
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    command:
      # Enable Docker in Traefik, so that it reads labels from Docker services
      - --providers.docker
      # Add a constraint to only use services with the label for this stack
      # from the env var TRAEFIK_TAG
      - --providers.docker.constraints=Label(`traefik.constraint-label-stack`, `${TRAEFIK_TAG?Variable not set}`)
      # Do not expose all Docker services, only the ones explicitly exposed
      - --providers.docker.exposedbydefault=false
      # Enable Docker Swarm mode
      - --providers.docker.swarmmode
      # Enable the access log, with HTTP requests
      - --accesslog
      # Enable the Traefik log, for configurations and errors
      - --log
      # Enable the Dashboard and API
      - --api
    deploy:
      placement:
        constraints:
          - node.role == manager
      labels:
        # Enable Traefik for this service, to make it available in the public network
        - traefik.enable=true
        # Use the traefik-public network (declared below)
        - traefik.docker.network=${TRAEFIK_PUBLIC_NETWORK?Variable not set}
        # Use the custom label "traefik.constraint-label=traefik-public"
        # This public Traefik will only use services with this label
        - traefik.constraint-label=${TRAEFIK_PUBLIC_TAG?Variable not set}
        # traefik-http set up only to use the middleware to redirect to https
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-https-redirect.redirectscheme.scheme=https
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-https-redirect.redirectscheme.permanent=true
        # Handle host with and without "www" to redirect to only one of them
        # Uses environment variable DOMAIN
        # To disable www redirection remove the Host() you want to discard, here and
        # below for HTTPS
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-http.rule=Host(`${DOMAIN?Variable not set}`) || Host(`www.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-http.entrypoints=http
        # traefik-https the actual router using HTTPS
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.rule=Host(`${DOMAIN?Variable not set}`) || Host(`www.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.entrypoints=https
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.tls=true
        # Use the "le" (Let's Encrypt) resolver created below
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.tls.certresolver=le
        # Define the port inside of the Docker service to use
        - traefik.http.services.${STACK_NAME?Variable not set}-proxy.loadbalancer.server.port=80
        # Handle domain with and without "www" to redirect to only one
        # To disable www redirection remove the next line
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-www-redirect.redirectregex.regex=^https?://(www.)?(${DOMAIN?Variable not set})/(.*)
        # Redirect a domain with www to non-www
        # To disable it remove the next line
        - traefik.http.middlewares.${STACK_NAME?Variable not set}-www-redirect.redirectregex.replacement=https://${DOMAIN?Variable not set}/$${3}
        # Redirect a domain without www to www
        # To enable it remove the previous line and uncomment the next
        # - traefik.http.middlewares.${STACK_NAME}-www-redirect.redirectregex.replacement=https://www.${DOMAIN}/$${3}
        # Middleware to redirect www, to disable it remove the next line
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-https.middlewares=${STACK_NAME?Variable not set}-www-redirect
        # Middleware to redirect www, and redirect HTTP to HTTPS
        # to disable www redirection remove the section: ${STACK_NAME?Variable not set}-www-redirect,
        - traefik.http.routers.${STACK_NAME?Variable not set}-proxy-http.middlewares=${STACK_NAME?Variable not set}-www-redirect,${STACK_NAME?Variable not set}-https-redirect

  db:
    image: postgres:12
    volumes:
      - app-db-data:/var/lib/postgresql/data/pgdata
    env_file:
      - .env
    environment:
      - PGDATA=/var/lib/postgresql/data/pgdata
    deploy:
      placement:
        constraints:
          - node.labels.${STACK_NAME?Variable not set}.app-db-data == true

  pgadmin:
    image: dpage/pgadmin4
    networks:
      - ${TRAEFIK_PUBLIC_NETWORK?Variable not set}
      - default
    depends_on:
      - db
    env_file:
      - .env
    deploy:
      labels:
        - traefik.enable=true
        - traefik.docker.network=${TRAEFIK_PUBLIC_NETWORK?Variable not set}
        - traefik.constraint-label=${TRAEFIK_PUBLIC_TAG?Variable not set}
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-http.rule=Host(`pgadmin.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-http.entrypoints=http
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-http.middlewares=${STACK_NAME?Variable not set}-https-redirect
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-https.rule=Host(`pgadmin.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-https.entrypoints=https
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-https.tls=true
        - traefik.http.routers.${STACK_NAME?Variable not set}-pgadmin-https.tls.certresolver=le
        - traefik.http.services.${STACK_NAME?Variable not set}-pgadmin.loadbalancer.server.port=5050

  queue:
    image: rabbitmq:3
    # Using the below image instead is required to enable the "Broker" tab in the flower UI:
    # image: rabbitmq:3-management
    #
    # You also have to change the flower command

  flower:
    image: mher/flower
    networks:
      - ${TRAEFIK_PUBLIC_NETWORK?Variable not set}
      - default
    env_file:
      - .env
    command:
      - "--broker=amqp://guest@queue:5672//"
      # For the "Broker" tab to work in the flower UI, uncomment the following command argument,
      # and change the queue service's image as well
      # - "--broker_api=http://guest:guest@queue:15672/api//"
    deploy:
      labels:
        - traefik.enable=true
        - traefik.docker.network=${TRAEFIK_PUBLIC_NETWORK?Variable not set}
        - traefik.constraint-label=${TRAEFIK_PUBLIC_TAG?Variable not set}
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-http.rule=Host(`flower.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-http.entrypoints=http
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-http.middlewares=${STACK_NAME?Variable not set}-https-redirect
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-https.rule=Host(`flower.${DOMAIN?Variable not set}`)
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-https.entrypoints=https
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-https.tls=true
        - traefik.http.routers.${STACK_NAME?Variable not set}-flower-https.tls.certresolver=le
        - traefik.http.services.${STACK_NAME?Variable not set}-flower.loadbalancer.server.port=5555

  backend:
    image: '${DOCKER_IMAGE_BACKEND?Variable not set}:${TAG-latest}'
    depends_on:
      - db
    env_file:
      - .env
    environment:
      - SERVER_NAME=${DOMAIN?Variable not set}
      - SERVER_HOST=https://${DOMAIN?Variable not set}
      # Allow explicit env var override for tests
      - SMTP_HOST=${SMTP_HOST}
    build:
      context: ./backend
      dockerfile: backend.dockerfile
      args:
        INSTALL_DEV: ${INSTALL_DEV-false}
    deploy:
      labels:
        - traefik.enable=true
        - traefik.constraint-label-stack=${TRAEFIK_TAG?Variable not set}
        - traefik.http.routers.${STACK_NAME?Variable not set}-backend-http.rule=PathPrefix(`/api`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)
        - traefik.http.services.${STACK_NAME?Variable not set}-backend.loadbalancer.server.port=80

  celeryworker:
    image: '${DOCKER_IMAGE_CELERYWORKER?Variable not set}:${TAG-latest}'
    depends_on:
      - db
      - queue
    env_file:
      - .env
    environment:
      - SERVER_NAME=${DOMAIN?Variable not set}
      - SERVER_HOST=https://${DOMAIN?Variable not set}
      # Allow explicit env var override for tests
      - SMTP_HOST=${SMTP_HOST?Variable not set}
    build:
      context: ./backend
      dockerfile: celeryworker.dockerfile
      args:
        INSTALL_DEV: ${INSTALL_DEV-false}

  frontend:
    image: '${DOCKER_IMAGE_FRONTEND?Variable not set}:${TAG-latest}'
    build:
      context: ./frontend
      args:
        FRONTEND_ENV: ${FRONTEND_ENV-production}
    deploy:
      labels:
        - traefik.enable=true
        - traefik.constraint-label-stack=${TRAEFIK_TAG?Variable not set}
        - traefik.http.routers.${STACK_NAME?Variable not set}-frontend-http.rule=PathPrefix(`/`)
        - traefik.http.services.${STACK_NAME?Variable not set}-frontend.loadbalancer.server.port=80

volumes:
  app-db-data:

networks:
  traefik-public:
    # Allow setting it to false for testing
    external: ${TRAEFIK_PUBLIC_NETWORK_IS_EXTERNAL-true}

It worked
thanks :)


Integration Test - Release - 2020/08/26 19:08:08

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

GitHub user PRMergerTest2 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


Unify the two login methods

Issue Name
Summary
maybe move to backend
Steps to Reproduce
Current Behaviour
Expected Behaviour
Extra Details
Here you should include details about the system (if it is unique) and possible information about a fix (feel free to link to code where relevant). Screenshots/GIFs are also fine here.

replace username burger menu ngif

Duplicate of #23


Encoding cleanup, adding event.DataEncoded flag, adding event.SetData

There was an issue developing where the data payload was un-trustable to the codecs. I needed a way to set the data and have it pass through the event data encoding, which is not the same as the transport encoding. So now there is a new method on event:
event.SetData(mydata)

And this results in the event data becoming encoded on the event according to the event's encoding settings.

Ready for another look, thanks!


Implement array.prototype.flat and flatMap

This PR implements the stage 3 proposal Array methods: Array.prototype.flat and Array.prototype.flatMap as JsBuiltIns.
Draft specification text is here: https://tc39.github.io/proposal-flatMap/
Notes:

3 test262 failures:

length is not configurable (for both methods), this is due to ScriptFunction lengths in CC all not being configurable and should be fixed if #5405 is merged
the case using null fails in this test: https://github.com/tc39/test262/blob/master/test/built-ins/Array/prototype/flatMap/thisArg-argument.js (the other cases in the file pass)

I can't find a feasible way to test the length is too big error - not sure if CC can actually handle arrays that big anyway so maybe not worth implementing that case?
Should probably have a few more CI tests

Fixes #5543

Regarding testing the length-too-long error; I'd be surprised if any engine today can hit that case... 2^53-1 is a lot of items! If each just took up 1 bit, you'd still need ~2^50 bytes of memory... I guess if you were dedicated enough and willing to use swap space then you could get there... But for now I think we're ok without a test for that specific case.

Regarding the failure with passing null as thisArg, looks like that might be due to a spec change; see the note at https://tc39.github.io/ecma262/#sec-function.prototype.call
@zenparsing do you know if binding null or undefined is supposed to keep that value as-is in the called function, rather than substituting the global object?

Oh; null and undefined aren't replaced if the function is a strict function. I see now that the test case says that it should only be run in strict mode, how were you running it @rhuanjl ? Were you using the test 262 test harness, or just running that one file?

I made my own mini-harness to run the test files. That case still failed with strict mode though - without strict mode most of the cases in the file failed, with strict mode only that case (the one with null) failed.

Apart from those small comments, this looks reasonable to me. Thanks for taking this on!
I'm not sure why the no_icu_static_ubuntu_linux_debug test is failing, it looks like somehow the two functions haven't been added to the array prototype, but in the normal static_ubuntu_linux_debug CI run the test did run and pass.

Ah, and just saw that windows 10 ci_lite_x64_debug also failed in the same way.

Do the windows lite build and ubuntu no_icu builds build without JsBuiltins? (these functions are the first JsBuiltins without a fall back C++ implementation so if JsBuiltins are disabled these won't work)

Thank you for the reviews @MSLaguana @boingoing I think I've fixed all the points you've raised - I'll hold off on squashing for now as I anticipate that some further reviews/changes will be required.
I'm nervous about non-icu/lite builds not supporting this - could be confusing for some users, should/could we either:
a) enable JsBuiltIns for non-icu/lite builds - they don't need icu etc. OR
b) put in a stub that throws a message saying that these methods aren't supported in a non-icu/lite build. OR
c) Make a c++ implentation

Lite build seems to work with the "normal" no-jit bytecode - tested offline and CI runs with it - as can be seen. I figure that a change of that kind should perhaps be a separate PR though? (I added it here just to test the idea)

Thanks for taking the time to implement this.
I took a lot at the JIT output and it seems to be doing a reasonable job, but recussions isn't handled very precisely when it comes to inlining. It'd be interesting to see how perf would compare to an implementation using a stack instead of recursion, but that's not strictly necessary for this PR.

@sigatrev Thanks for the review. I've:

removed the IsCallable function AND
as a separate commit I've refactored FlattenIntoArray to use a stack - kept as a separate commit so can be easily dropped if not wanted.
I've added an extra loop to FlattenIntoArrayMapped so it no longer has to call FlattenIntoArray

@rhuanjl I did some microbenchmarking and the original recursive case runs ~30% faster. Thanks for giving that a shot, and the quick turn-around.

OK, I've updated to remove the stack based mechanism, I've kept the inner loop for FlattenIntoArrayMapped though as I assume that has minimal performance impact and if anything makes the code clearer.

@dotnet-bot test OSX osx_osx_debug_static please

@dotnet-bot test OSX static_osx_osx_debug please

Please let me know what's needed for this to be merged.
I assume it will require a GUID update as it changes the byte code, anything else?

@rhuanjl I think this is pretty much ready. I should be able to get it merged in the next couple of days, sorry for the delay.

@rhuanjl can you enable us to update the PR so I update bytecode headers and GUID?

@sigatrev Allow edits from maintainers is already ticked not sure if there's anything else I can do to give you additional access?
(let me know if you'd like me to regen the byte code)

Thanks for testing and merging this @sigatrev
And thanks everyone for the reviews.

@rhuanjl Thanks for your patience. We really appreciate the contributions.


Collapsed menu is broken (no background)

Hi,
I'm using the Ruby theme and the collapsed menu (on mobile for example) is broken. The menu has no background.
It's also broken on the live demo: https://ruby.ghost.io/

I can confirm, the same thing is happening when I try to use this theme on the latest version of Ghost.

Thanks everyone for the feedbacks! This is now fixed by adding default background color.

Thanks for the quick fix!


i18n part 2: Swap constants in the auth feature folder to use i18n JSON resources

As in title, as well as:

Tweaked the i18n setup, so it only loads the "global" namespace by default. This is where generically shared strings like "Continue", "Submit" etc are contained
Restructure of the translation key for the auth feature area from the literal constant file conversion to something more resilient to changes where possible. There are some messages I can't think of a better name for, so those I have left more or less the literal value as the key (but maybe still moved it within an object named after the page/form for better organisation)

Still WIP to fix the billion of tests I have broken because of the constant removals since they are used in tests too...

Web frontend checklist

[x] Formatted my code with yarn format && yarn lint --fix
[x] There are no warnings from yarn lint
[x] There are no console warnings when running the app
[x] Added any new components to storybook
[ ] Added tests where relevant
[x] All tests pass
[x] Clicked around my changes running locally and it works
[ ] Checked Desktop, Mobile and Tablet screen sizes

Moved a few things around, the main change being the tests now uses a separate i18n.init with all of the resources loaded in rather than doing so lazily via dynamic import like in the main app. This means I don't have to convert all of the renders to be async to do
  await waitFor(() => {
    expect(i18n.isInitialized).toBe(true);
  });

in a custom render that wraps React Testing Library's render, which was a bit awkward as it meant I had to update every test to add await in front of every render in every test suite... reverting that was not fun either 😢
Otherwise, for some strange reasons I couldn't use the useTranslation hook in auth store without most tests being stuck in the Suspense fallback loading state. It's super weird and I still don't know why. I think it's ok though since in the tests, we are more interested in the translations being applied in the UI properly rather than testing specifically the Suspense behaviour, which should be tested by react-i18next (I hope) and so shouldn't be our concern to test that.


e2e test are stopped working - issue with selenium-webdriver

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[ x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

BUG REPORT

What is the current behavior?

e2e is not running

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

npm run e2e

What is the expected behavior?

it should run functional tests

What is the motivation / use case for changing the behavior?

/angular2-webpack-starter/node_modules/selenium-webdriver/error.js:27
super(opt_error);
^
SessionNotCreatedError: session not created exception
from unknown error: Runtime.executionContextCreated has invalid 'context': {"auxData":{"frameId":"17948.1","isDefault":true},"id":1,"name":"","origin":"://"}
(Session info: chrome=54.0.2840.16)

Please tell us about your environment:
node version 6.5
Chrome Version 54.0.2840.16 beta (64-bit)

Angular version: 2.0.0-beta.X
Browser: [all | Chrome Version 54.0.2840.16 beta (64-bit) | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)
have tried to update webdriwer by running:
npm i webdriver-manager update --chromedriver
npm run e2e stopped working

See my comments: here


Support QDM::FamilyHistory

Pull requests into cqm-parsers require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[ ] This pull request describes why these changes were made.
[ ] Internal ticket for this PR:
[ ] Internal ticket links to this PR
[ ] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Codecov Report

Merging #19 into full_qdm_5_4_support will increase coverage by 2.35%.
The diff coverage is 100%.

@@                   Coverage Diff                    @@
##           full_qdm_5_4_support      #19      +/-   ##
========================================================
+ Coverage                 85.81%   88.17%   +2.35%
========================================================
  Files                        56       67      +11
  Lines                      1361     1641     +280
========================================================
+ Hits                       1168     1447     +279
- Misses                      193      194       +1

Impacted Files
Coverage Δ

.../data-element-importers/family_history_importer.rb
100% <100%> (ø)

lib/qrda-import/patient_importer.rb
93.33% <100%> (+0.83%)
:arrow_up:

lib/cqm_report.rb
100% <100%> (ø)
:arrow_up:

lib/qrda-export/catI-r5/qrda1_r5.rb
100% <100%> (ø)
:arrow_up:

...element-importers/assessment_performed_importer.rb
94.11% <0%> (-5.89%)
:arrow_down:

lib/qrda-export/helper/patient_view_helper.rb
76.92% <0%> (-5.43%)
:arrow_down:

...ment-importers/physical_exam_performed_importer.rb
95% <0%> (-5%)
:arrow_down:

...nt-importers/laboratory_test_performed_importer.rb
95.45% <0%> (-4.55%)
:arrow_down:

...t-importers/diagnostic_study_performed_importer.rb
96% <0%> (-4%)
:arrow_down:

...-element-importers/procedure_performed_importer.rb
96.15% <0%> (-3.85%)
:arrow_down:

... and 27 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2ba745...6888716. Read the comment docs.


Information architecture and design

Mirroring the API is a good way to go. Here’s the root API page on YNMP:
http://yournextmp.popit.mysociety.org/api/v0.1/


Initial Written Announcement of GC Design System w/ DTO

Send notice via:

CDS Blog
DTO Blog
design-GC-conception Slack
GC Web Comms Slack
GC Web Manager Newsletter
PP Newsletter

Initial DTO/CDS Merger announcement via Blog tentatively scheduled for May 14
DTO / GC Design System DS related Blog tentatively scheduled for May 21


sysfs: add parsing of clocksource

Read current and available clocksource data from
/sys/devices/system/clocksource
Related to https://github.com/prometheus/node_exporter/issues/1336

But LGTM beside the naming.

@discordianfish Does this look ok now?

Great, LGTM!


Refactor Colwise

Perfect. Thanks for double checking. Feel free to merge it. :)
Em sex., 23 de set. de 2022 08:07, Elias Carvalho @.***>
escreveu:

@.**** commented on this pull request.
In src/transforms.jl
https://github.com/JuliaML/TableTransforms.jl/pull/129#discussion_r978519192
:

@@ -273,10 +283,10 @@ function applyfeat(transform::Colwise, feat, prep)

newfeat = 𝒯 |> Tables.materializer(feat)
cache values for each column

fcache = last.(vals)

In this case, fcache is the tuple of colcaches and snames:
fcache = (caches, snames)
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaML/TableTransforms.jl/pull/129#discussion_r978519192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3JPTAQX7OHA6H4K2QTV7WFQZANCNFSM6AAAAAAQTIH35A
.
You are receiving this because your review was requested.Message ID:
@.***>


Adding new variable "cleanup_use_sudo"

We faced this in the past where we needed to run the cleanup command as sudo as all our releases are owned by a different user than the one which we use for deployment.

I don't this what it is save. What do you think?

Sorry but what do you mean by "I don't this what it is save."?

Do you ask password on sudo?

Nope.

So every body can delete files from sudo as root?


RightOn Mobile UI: Phase 2 Wrong Answer Info Carousel

Currently the wrong answer explanations are separated in individual cards with proper titles, but need to get them separated to be "swipable" left and right. Each one needs to fill the the screen individually
Seems like a Carousel indexing issue perhaps?

Phase 2 Source code
Component rendering the carousel Component

https://github.com/rightoneducation/righton-app/pull/407


Text in tabs can get truncated vertically

Screenshot (see the word 'original').

My environment
Boxy Theme: 5.2.0
Sublime Text: 3126
If you can give me an easy way to dump the more specific, relevant theme settings, then I'll do it.
OS: Linux Mint 18.1 Serena x86_64
Kernel: 4.10.0-20-generic
Shell: /bin/bash
Resolution: 1280x800
DE: Cinnamon 3.2.7
WM: Mutter (Muffin)
WM Theme: Cinnamon (SimpleX-Grayscale_NJ)
Theme: Mint-Y [GTK2/3]
Icons: Paper [GTK2/3]
Font: Noto Sans UI 11 [GTK2/3]

@ACinnamonUserInUK Hello! That's Sublime Text issue. It was fixed in 3127+.

Thanks. I discover that that build is available from here: www.sublimetext.com/3dev and not from www.sublimetext.com/3.


Component type of icon prop type support

Hi there,
If this component plugin depends on the font - FontAwesome which comes from the repository : react-native-vector-icons.  Why not support component type for the 'icon' prop directly?
eg:
import ItemCell from 'react-native-item-cell';
import Icon from 'react-native-vector-icons/FontAwesome';
<ItemCell
 icon={<Icon name="rocket"/>}
>
  Item
</ItemCell>
#just a feature wanted

Filed a PR: https://github.com/APSL/react-native-item-cell/pull/21
Please have a review and fell free to handle it. Thanks


Make queue table be sorted by machine.

Alternatively, make separate queue table for each goody. Might not be exactly what users want, since this way they wouldn't know where to look for queue.
Better just graphically separate those on a single table.


chore: update dependencies

Description
Updates a set of dependencies that were flagged by npm audit, as well as dependencies preventing builds/CI from working in node 12.
Removes an old (I think unused) gulp command that did stats on our generated CSS, as the dependency used was super old and causing problems. I did not see alternatives to the package, but since it was only used in 1 gulp task that I don't think has been used/touched in years I just removed.
If we think that gulp task is important I can look at forking/updating the package.
Types of changes

[ ] Documentation
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (anything that would change an existing functionality)
[x] Maintenance (refactor, code cleaning, development tools...)

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] The pull request title and template are correctly filled.
[x] The pull request targets the right branch (develop or develop-v...).
[x] My commits are correctly titled and contain all relevant information.
[ ] I have updated the documentation accordingly to my changes (if relevant).
[ ] I have added tests to cover my changes (if relevant).

Sigh this now fails because some of the dependencies no longer support node 6. Given Node 6 is no longer a supported/maintained release (EOL was May 2019) how do we feel about ditching it?

Given Node 6 is no longer a supported/maintained release (EOL was May 2019) how do we feel about ditching it?

We can drop support for NodeJS 6.
Supporting 8, 10 and 12 is ok.

But we should mention this in the changelog then.
Also do not forget to update engines in package.json and also the npm shrinkwrap and lock-file.

@DanielRuf I've been out of the loop for a long time; do we have a way we're recording changelog relevant notes? A tag on the PR, or a doc/file somewhere I'm not seeing?

I've added milestone as I'm pretty sure it's helpful to see what's in the release

@kball we have the commit lint rules (chore, fix, refactor, docs, ...: title)


[Event Request] Page296 "Recurring Req. Worksheet"  – Action "Carry &Out Action Message"

[Event Request] Page296 "Recurring Req. Worksheet"  – Action "Carry &Out Action Message"
Could we please have a in publisher OnBeforeCarryOutActionMessage in Action "Carry &Out Action Message" Page296 "Recurring Req. Worksheet"?
                action("Carry &Out Action Message")
                {
                    ApplicationArea = Planning;
                    Caption = 'Carry &Out Action Message';
                    Ellipsis = true;
                    Image = CarryOutActionMessage;
                    ToolTip = 'Use a batch job to help you create actual supply orders from the order proposals.';

                    trigger OnAction()
                    var
                        MakePurchOrder: Report "Carry Out Action Msg. - Req.";
                        // >>>>>>>>
                        IsHandled: Boolean;
                    	//stop KVSKBA
                    begin
                        // >>>>>>>>
                        OnBeforeCarryOutActionMessage(Rec, IsHandled);
                        if not IsHandled then begin
                            MakePurchOrder.SetReqWkshLine(Rec);
                            MakePurchOrder.RunModal;
                            MakePurchOrder.GetReqWkshLine(Rec);
                        end;
                        // <<<<<<<<
                        CurrentJnlBatchName := GetRangeMax("Journal Batch Name");
                        CurrPage.Update(false);
                    end;
                }

..
   // >>>>>>>>
 [IntegrationEvent(false, false)]
 local procedure OnBeforeCarryOutActionMessage(var RequisitionLine: Record "Requisition Line"; var IsHandled: Boolean)
 begin
 end;
   // <<<<<<<<

hi @ttaeufer . I would suggest you to hide our action, add your own action and run it in dependency on your required properties.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Gitea 1.3.0 with SSH server enabled crashes with SIGSEGV when pushing to SSH

Gitea version (or commit ref): 1.3.0
Git version: 2.15.0
Operating system:

Client: ArchLinux
Server: Debian Stretch

Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist: there it is

Description
When I want to push with git push to the included SSH server, Gitea crashes with a SIGSEGV. This crash is easily reproductible.
How to reproduce

Download and install a copy of Gitea 1.3.0 to upgrade an older version
Start it with gitea web with embeded SSH server enabled
git push commits to remote. It should crash with a SIGSEGV

Screenshots
Not revelant

Looks like https://github.com/go-gitea/gitea/issues/2885

#2923 need to be back port to v1.3 @lafriks

@lunny yes, I will do it


Use Random Ports for Kubo Node

Scramble the ports on at least init because it doesn't really matter what they are and it'd be nice to avoid conflicts.

Configuration info for what we listen on: https://github.com/ipfs/kubo/blob/master/docs/config.md#addresses
We'll need to change the following:
API - 5001 (random #)
Swarm - 4001 (random #)
Gateway - 8080 (null, disabled)
API / Swam ports should be fetchable from the JS side.


Fix for issue1030

Find the issue here https://github.com/fossasia/open-event-android/issues/1030

@the-dagger closing this. Will look as per requirements.


Screens for Permissions

[ ] Create
[ ] Read
[ ] Update
[ ] Delete

This must include the possibility to assign (and remove) Roles (see #44) to Permissions.

@ChrEggenberger the ODataController for Permissions is now available


False positive with MagicNumber on default value for parameters

Correct me if I'm wrong but the following method signature gets flagged with a MagicNumber with Detekt 1.0.0.RC8:
fun Context.vibrate(duration: Long = 300L)

I thought that default parameters won't be treated as such code smells. As for the configuration I'm using:
failFast: true

potential-bugs:
  UnsafeCast:
    active: false # We know what we're doing.
  LateinitUsage:
    active: false # We know what we're doing.

naming:
  FunctionMinLength:
    active: false
  FunctionMaxLength:
    active: false

style:
  TopLevelPropertyNaming:
    active: false
  MaxLineLength:
    active: false
  UnnecessaryParentheses:
    active: false # False positives.

complexity:
  LongParameterList:
    threshold: 8

comments:
  UndocumentedPublicClass:
    active: false
  UndocumentedPublicFunction:
    active: false

Detekt hasn't considered this case yet. MagicNumber allows constant declarations in constructors, but as it looks like not in functions.
We could need some help here. Would you mind taking a look at this @vanniktech ?


Can't sell full set after buying full set

The button's greyed out. Happened on market 178.

fixed on staging


Change DeadlyReentry.netkan to adapt to new archive directory structure

Yay for folder structure changes~~

:+1:


Instructions to use extension on a CRA project without adding files to the source code

The extension does not work with CRA.
Adding these commands to settings.json makes both Run and Debug to work.

You are saying it does not work with CRA, but I'm pretty sure it does work with the provided information.
Didn't it work for you when putting 'npm run test --' in the jest.command?
Your PR also removes important information from the Readme. Which is needed to debug JSX/TSX. Even if this is not connected to the extension. Many don't know it and will blame the extension.

Run works after adding "jestrunner.jestCommand": "npm run test --", same instructions are in the commit.
Debug was not working for me even after adding babel.config.js and jest config.
Also, this solution does not add new files to the source code.
I can leave in info about babel and jest config as another option.

I think the current README covers how to run and debug with CRA, it looks like the content was added in this PR:
https://github.com/firsttris/vscode-jest-runner/commit/0fd4dcc2cb39aa6eb00238215c749d820e26e001
Please reopen this if you think otherwise 👍


Altering captured reference damages spark results.

If you use a variable in dplyr::mutate() against a sparklyr data source the lazy eval captures references to user variables.  Changing values of those variables implicitly changes the mutate and changes the values seen in the sparklyr result (which is itself a query).  This can be worked around by dropping in dplyr::compute() but it seems like it can produce a lot of incorrect calculations.  Below is a small example and a lot information on the versions of everything beeing run.  I am assuming the is a sparklyr issue as the query views are failrly different than a number of other dplyr structures, but it could be a dplyr issue.
OSX 10.11.6. Spark installed as described at http://spark.rstudio.com
library('sparklyr')
spark_install(version = "2.0.0")

library('dplyr')
 #
 #  Attaching package: 'dplyr'
 #  The following objects are masked from 'package:stats':
 #
 #      filter, lag
 #  The following objects are masked from 'package:base':
 #
 #      intersect, setdiff, setequal, union
library('sparklyr')
R.Version()$version.string
 #  [1] "R version 3.3.2 (2016-10-31)"
packageVersion('dplyr')
 #  [1] '0.5.0'
packageVersion('sparklyr')
 #  [1] '0.5.2'
my_db <- sparklyr::spark_connect(version='2.0.0', master = "local")
class(my_db)
 #  [1] "spark_connection"       "spark_shell_connection" "DBIConnection"
my_db$spark_home
 #  [1] "/Users/johnmount/Library/Caches/spark/spark-2.0.0-bin-hadoop2.7"
print(my_db)
 #  $master
 #  [1] "local[4]"
 #
 #  $method
 #  [1] "shell"
 #
 #  $app_name
 #  [1] "sparklyr"
 #
 #  $config
 #  $config$sparklyr.cores.local
 #  [1] 4
 #
 #  $config$spark.sql.shuffle.partitions.local
 #  [1] 4
 #
 #  $config$spark.env.SPARK_LOCAL_IP.local
 #  [1] "<IP_ADDRESS>"
 #
 #  $config$sparklyr.csv.embedded
 #  [1] "^1.*"
 #
 #  $config$`sparklyr.shell.driver-class-path`
 #  [1] ""
 #
 #  attr(,"config")
 #  [1] "default"
 #  attr(,"file")
 #  [1] "/Library/Frameworks/R.framework/Versions/3.3/Resources/library/sparklyr/conf/config-template.yml"
 #
 #  $spark_home
 #  [1] "/Users/johnmount/Library/Caches/spark/spark-2.0.0-bin-hadoop2.7"
 #
 #  $backend
 #          description               class                mode                text              opened
 #  "->localhost:58964"          "sockconn"                "wb"            "binary"            "opened"
 #             can read           can write
 #                "yes"               "yes"
 #
 #  $monitor
 #          description               class                mode                text              opened
 #  "->localhost:58961"          "sockconn"                "rb"            "binary"            "opened"
 #             can read           can write
 #                "yes"               "yes"
 #
 #  $output_file
 #  [1] "/var/folders/7q/h_jp2vj131g5799gfnpzhdp80000gn/T//Rtmpxkn3Al/filefeb634813b11_spark.log"
 #
 #  $spark_context
 #  <jobj[5]>
 #    class org.apache.spark.SparkContext
 #    org.apache.spark.SparkContext@73d0c0e5
 #
 #  $java_context
 #  <jobj[6]>
 #    class org.apache.spark.api.java.JavaSparkContext
 #    org.apache.spark.api.java.JavaSparkContext@4e527f99
 #
 #  $hive_context
 #  <jobj[9]>
 #    class org.apache.spark.sql.SparkSession
 #    org.apache.spark.sql.SparkSession@1346ee7
 #
 #  attr(,"class")
 #  [1] "spark_connection"       "spark_shell_connection" "DBIConnection"

Expected outcome: s1 has the same value
Observed outcome: changing varaible v changes s1 column.

support <- copy_to(my_db,
                   data.frame(year=2005:2010),
                   'support')
v <- 0
s1 <- dplyr::mutate(support,count=v)

print(s1) # print 1
 #  Source:   query [6 x 2]
 #  Database: spark connection master=local[4] app=sparklyr local=TRUE
 #
 #     year count
 #    <int> <dbl>
 #  1  2005     0
 #  2  2006     0
 #  3  2007     0
 #  4  2008     0
 #  5  2009     0
 #  6  2010     0

# s1 <- dplyr::compute(s1) # likely work-around
v <- ''

print(s1) # print 2
 #  Source:   query [6 x 2]
 #  Database: spark connection master=local[4] app=sparklyr local=TRUE
 #
 #     year count
 #    <int> <chr>
 #  1  2005
 #  2  2006
 #  3  2007
 #  4  2008
 #  5  2009
 #  6  2010

Notice s1 changed its value (like due to lazy evaluation and having captured a reference to v).
version
 #                 _
 #  platform       x86_64-apple-darwin13.4.0
 #  arch           x86_64
 #  os             darwin13.4.0
 #  system         x86_64, darwin13.4.0
 #  status
 #  major          3
 #  minor          3.2
 #  year           2016
 #  month          10
 #  day            31
 #  svn rev        71607
 #  language       R
 #  version.string R version 3.3.2 (2016-10-31)
 #  nickname       Sincere Pumpkin Patch

Thanks for the reproducible bug report!
It looks to me like this is a bug in dplyr -- the lazy evaluation performed doesn't capture the value of v when the query is generated, and so every time the query is 'rendered' it looks up what the current value of v is.

Sounds good, I have copied the content over to https://github.com/hadley/dplyr/issues/2455 .

Look like that it this is known in https://github.com/hadley/dplyr/issues/2370 and fixed in the dev versio of dplyr.