Notification not displaying on iOS

Version Number of Plugin: 2.3.5
Device Tested On: iPhone 7 Plus
Simulator Tested On: iPhone XS Max
Version of VS: 7.7.2
Xamarin Forms: 3.4.0
Steps to reproduce the Behavior
Send notification to iOS devices from Firebase console
Send notification using Postman
Expected Behavior
Notification to appear on iOS
Actual Behavior
iOS receives notification (AFAIK) but does not display it. Postman POST doesn't give any errors.
I get this in the output:

5.8.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

The IMessagingDelegate protocol defines this member:
[ProtocolMember(IsRequired = false, IsProperty = false, IsStatic = false, Name = "DidReceiveMessage", Selector = "messaging:didReceiveMessage:", ParameterType = new[] { typeof(Messaging), typeof(RemoteMessage) }, ParameterByRef = new[] { false, false })]

But in the iOS project, the FirebasePushNotificationManager method DidReceiveMessage does not match the protocol member's signature, and is not exported. If I add an overload to this method with the same parameters and export it, I am able to receive notifications by setting a breakpoint in my code where I subscribe to the CrossFirebasePushNotification.Current.OnNotificationReceived event.
        [Export("messaging:didReceiveMessage:")]
        public void DidReceiveMessage(Messaging messaging, RemoteMessage remoteMessage) {
            DidReceiveMessage(remoteMessage.AppData);
        }

However there are no visible notifications displayed on the device. I'm not sure if this is due to some other reason. Any ideas?
I also noticed that the packages used in this plugin is a bit outdated. The protocol has changed in the later version.

Hi folks,
I finally fixed this error. I got notification displayed but no vibration. Don't know why, but its not that mendatory, as notification displays well.
For that I changed the value of FireBaseAppDelegateProxyEnabled to TRUE and I tested a lot of things in the AppDelegate.cs with a solution.
<key>FirebaseAppDelegateProxyEnabled</key>
	<true/>

In the AppDelegate.cs :
Old version with bug
        public override void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken)
        {
#if DEBUG
            FirebasePushNotificationManager.DidRegisterRemoteNotifications(deviceToken);
#endif
#if RELEASE
		                FirebasePushNotificationManager.DidRegisterRemoteNotifications(deviceToken,FirebaseTokenType.Production);
#endif
        }

        public override void FailedToRegisterForRemoteNotifications(UIApplication application, NSError error)
        {
            FirebasePushNotificationManager.RemoteNotificationRegistrationFailed(error);
        }

        public override void DidReceiveRemoteNotification(UIApplication application, NSDictionary userInfo, Action<UIBackgroundFetchResult> completionHandler)
        {
            FirebasePushNotificationManager.DidReceiveMessage(userInfo);
        }

        public override void OnActivated(UIApplication uiApplication)
        {
            FirebasePushNotificationManager.Connect();
            FacebookClientManager.OnActivated();
            base.OnActivated(uiApplication);

        }

        public override bool OpenUrl(UIApplication app, NSUrl url, NSDictionary options)
        {
             FacebookClientManager.OpenUrl(app, url, options);
             return true;
        }

        public override bool OpenUrl(UIApplication application, NSUrl url, string sourceApplication, NSObject annotation)
        {
            FacebookClientManager.OpenUrl(application, url, sourceApplication, annotation);
            return true;
        }

        public override void DidEnterBackground(UIApplication application)
        {
            FirebasePushNotificationManager.Disconnect();
        }

New fixed :
public override void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken)
        {
            FirebasePushNotificationManager.DidRegisterRemoteNotifications(deviceToken);
        }

        public override void FailedToRegisterForRemoteNotifications(UIApplication application, NSError error)
        {
            FirebasePushNotificationManager.RemoteNotificationRegistrationFailed(error);
        }
        public override void DidReceiveRemoteNotification(UIApplication application, NSDictionary userInfo, Action<UIBackgroundFetchResult> completionHandler)
        {
            FirebasePushNotificationManager.DidReceiveMessage(userInfo);
        }

        public override void OnActivated(UIApplication uiApplication)
        {
            //FirebasePushNotificationManager.Connect();
            FacebookClientManager.OnActivated();
            base.OnActivated(uiApplication);

        }

        public override bool OpenUrl(UIApplication app, NSUrl url, NSDictionary options)
        {
             FacebookClientManager.OpenUrl(app, url, options);
             return true;
        }

        public override bool OpenUrl(UIApplication application, NSUrl url, string sourceApplication, NSObject annotation)
        {
            FacebookClientManager.OpenUrl(application, url, sourceApplication, annotation);
            return true;
        }

        //public override void DidEnterBackground(UIApplication application)
        //{
        //    // Use this method to release shared resources, save user data, invalidate timers and store the application state.
        //    // If your application supports background exection this method is called instead of WillTerminate when the user quits.
        //    FirebasePushNotificationManager.Disconnect();
        //}

@Umar3x this does not solve the issue for me.
There is a lot of code in your comment, so please correct me if I've missed anything. But from what I see you have done four changes. Removed the calls to Connect and Disconnect, and removed the call to the deprecated overload DidRegisterRemoteNotifications(deviceToken,FirebaseTokenType.Production); which manually sets the APN-token and what type of token this is.
You also set the FirebaseAppDelegateProxyEnabled key to true in Info.plist. What this does is enable method swizzling in obj-c so that firebase handles setting the APN-token itself. I think this means that you can also comment out the call to FirebasePushNotificationManager.DidRegisterRemoteNotifications in your override method. When the key is set to false, method swizzling is disabled and you have to set the APN-key manually.
I have tested changes you propose, but unfortunately I'm still getting the same error in my device log output:

5.4.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

I'm not sure which parameters you are referring to in your last sentence, can you show an example?

@KristofferBerge
Sorry to hear that. Yep you are right on everything, and I commented out the DidRegisterRemoteNotifications. Just to be sure you got every elements, I also updated to latest version 2.3.5 as I were on 2.3.0 before.
I think I missed one important change I've made to make it work about the notification .json. We have a PHP backend (Symfony) that handles push notification. In the Push's .json we had to add this few properties :
"priority":"high","content_available":true,"sound":"default"
About my last sentence, what I meant is that there have been a major update about notification handling in latest iOS (12) that reset to default a lot of things for notifications.
Parameters => Notifications => YourApp.

I did not notice that my test iPhone had by default a notification behavior for my app that was to show notification only on LockScreen w/o sound or banner.
Plus you got an extra menu "Options" at the bottom of this page where you can choose how to preview notifcations, that, was by default too set to "If unlocked" => put it to Always .

I'm sending test-notifications from the firebase console, so I would guess that the payload is correct.
Ok now I understand. The notification settings for the application is as it should be. All choices are selected and show is set to always.

@rdelrosario Any suggestions here? This is still an issue

@KristofferBerge
You still do not receive notification on iOS ? Even when app is closed, nothing received ?

As I explained in my previous posts, the app receives the notification, but an error is logged out in the device output. The notification never reaches my application code.

5.8.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

@KristofferBerge just reviewed this conversation. One question in my mind ... do you use APNs certificate or APNs Auth key ?

Still an issue for me. I use APNs certificate

What is the solution, to work in a stable way, without depending on the iOS version.?
I have the same error with:
FirebaseAppDelegateProxyEnabled: true
$ sudo idevicesyslog | grep "Firebase"
5.8.0 - [Firebase/Messaging][I-FCM012002] Error in application:didFailToRegisterForRemoteNotificationsWithError: no se encontr\M-C\M-3 ninguna cadena de autorizaci\M-C\M-3n \M-b\M^@\M^\aps-environment\M-b\M^@\M^] para la app

5.8.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

Hello all, does anybody solve this problem? Same is for me too.

Anyone managed to resolve this issue?


Animation for the selected tab

Would you mind to add the animation changing the position of underline of the selected tabs? For the Obj-C library uacaps/PageMenu, there is an option called scrollAnimationDurationOnMenuItemTap that can control it. I hope you can add this animation effect to the library. Thanks.

Alright, I'm working on it!

Please use version 0.4.5! Let me know if there is still something wrong.

It works. Thank you so much.

:smile:


fix some mistake

Fix some minor bugs.

/assign @bprashanth Please review, Thanks.

@lavalamp  Are there any problems with these commit？


how to download data from IEEE or google scholar for a specific topics in CSV format

how to download data from IEEE or google scholar for a specific topics in CSV format

On IEEE Xplore you can perform your search and then click the "Export" button to download a CSV file.

Thank you for your reply it helps me a lot, I was wondering what will be the solution for google scholar

Not sure about how to do it with Scholar. You can export single entries from a search in BibTex format and import them I believe.

Unfortunately, Google Scholar is closed off and export or scraping the results is not made easy. As mentioned by @isazi, the only solution would be to export the result as BibTex and then use litstudy.load_bibtex to import the results. Alternatively, you can write down the DOIs of the articles and use litstudy.fetch_semanticscholar or litstudy.fetch_crossref to import them (see [1])
[1] https://nlesc.github.io/litstudy/api/sources.html


[PBIOS-328] Collapsible Docs

What does this PR do?
[PBIOS-328] Collapsible Docs
How to test? Steps to confirm the desired behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See addition/change

Checklist

[x] LABELS - Add a label: breaking, bug, improvement, documentation, or enhancement. See Labels for descriptions.
[x] RELEASES - Add the appropriate label: Ready for Testing / Ready for Release
[x] TESTING - Have you tested your story?

The examples are not exposing the Collapsible component but the CollapsableDoc. One suggestion would be to add an example showing what is the Collapsable doc or changing the Catalog to expose the Collasable component instead od the Collapsable doc.


Bug: props mapRecordsToProps are not updated when returning empty object

const mapRecordsToProps = ({ type, id }) => {
    if (id) {
      return { record: q => q.findRecord({ type, id }) }
    }

    return {}
  }

If initially provided with id, record is found and passed to props. If subsequently no id is provided and we return an empty object, the record prop remains intact as is.

@Michiel87 fixed in https://www.npmjs.com/package/react-orbitjs/v/0.1.12


Add inlined schema()

Added a second way to call the schema function with a type parameter.

Thank you!


feat: get theme author profile picture

Description
As of currently, profile pictures of github users aren't stored in the themes github repo, so I accessed github's own cdn at https://avatars.githubusercontent.com/ .
Closes #28
What change does this PR introduce?
Please select the relevant option(s).

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] CI/CD (updates related to the CI/CD process)
[ ] Documentation update (changes to docs/code comments)
[ ] Chore (miscellaneous tasks that do not fall into the above options)

What is the proposed approach?
Using github's own cdn, we can make sure that if a user changes his profile picture our website will be up to date.
Checklist:

[x] The commit message follows our adopted guidelines

Fix added and works as expected

LGTM, thanks!


Handle missing elevation data

Heightgraph currently doesn't handle cases where part or all of the data points have coordinates, but no elevation/altitude value.
This results in incomplete graphs and NaN errors/warnings, see comparison below and the Plunker console:

current
expected

Plunker
Plunker, see also Elevation#84

Diagrams shown are variants of the same elevation profile with missing values at different positions:

original
middle (left of second segment)
left
right
all missing

Reasons for missing elevation data e.g. are using CGIAR CSI SRTM that has no data beyond 60 degrees north and south and outside shorelines, see abrensch/brouter#137, nrenner/brouter-web#203.
The suggested solution would be to show missing elevation values as gaps by using a d3 defined accessor function, e.g. here:
d3Area().defined(d => d.altitude !== undefined && d.altitude !== null)

I guess I could provide a PR if no one else feels like it.

Have you considered filling emtpy slots on your side? That way you could interpolate the elevation or fill it with 0 etc., depending on your needs.
I am not sure if this plugin should fix "incorrect"/"missing" data?

A corresponding issue is Elevation #63 with a few upvotes and another PR, so we're not the only ones.
While interpolation or assuming 0 might work for some cases, it will be misleading or just wrong for others. I personally would expect missing elevation data to be reflected in the graph as such.

Setting it to 0 would result in the same rendering result as you shown above wouldn't it?
The difference would then be that when hovering over the entry that instead of showing 0 there would be something like "not defined"? That said, feel free to propose a PR and check back with @TheGreatRefrigerator if this features is desired for this repo.

Setting it to 0 would result in the same rendering result as you shown above wouldn't it?

Not quite. The connection would be different and rest would only look the same because this artificial example has a min height of 0 and I disabled extending the elevations bounds for better comparison.
With a more realistic example the differences are:

the graph outline is flattened when the elevation range is extended to 0
defined cuts the graph off with a vertical line, while with 0 it connects normally with a diagonal line to the next data point
due to elevation bounds extension by 10%, the graph visibly continues at height 0, making the filling look like an actual height of 0

0 filling:

defined:


Add Complex Types to What's New in EF8

I wrote a section on the Math improvements in RC1 if you want to add it to this (and possibly the blog post) for tomorrow:

Enhancements to Math
Generic math interfaces were introduced in .NET 7. Concrete types like double and float implemented these interfaces adding new APIs mirroring the existing functionality of Math and MathF.
EF Core 8 translates calls to these generic math APIs in LINQ using providers' existing SQL translations for Math and MathF. This means you're now free to choose between calls either like Math.Sin or double.Sin in your EF queries.
We worked with the .NET team to add two new generic math methods in .NET 8 that are implemented on double and float. These are also translated to SQL in EF Core 8.

.NET
SQL

DegreesToRadians
RADIANS

RadiansToDegrees
DEGREES

Finally, we worked with Eric Sink in the SQLitePCLRaw project to enable the SQLite math functions in their builds of the native SQLite library. This includes the native library you get by default when you install the EF Core SQLite provider. This enables several new SQL translations in LINQ.

.NET
SQLite

DegreesToRadians
radians

RadiansToDegrees
degrees

Acos
acos

Acosh
acosh

Asin
asin

Asinh
asinh

Atan
atan

Atan2
atan2

Atanh
atanh

Ceiling
ceiling

Cos
cos

Cosh
cosh

Exp
exp

Floor
floor

Log
ln (or log depending on overload)

Log2
log2

Log10
log10

Pow
pow

Sign
sign

Sin
sin

Sinh
sinh

Sqrt
sqrt

Tan
tan

Tanh
tanh

Truncate
trunc

@bricelam Thanks! I've added it here, but I'll leave the blog post focused on complex types.


motionSensors.orientation doesn't differentiate between pointing down or up

When using motionSensors.orientation or motionSensors.absoluteOrientation to get pitch, the value is the same if I move the device down or up. Therefore you cannot differentiate from up and down.
Measurements on ios:
device pointing up: pitch = 0
device pointing a little bit up: pitch = 1
device pointing straight, pitch = 1.57
device pointing a little bit down: pitch = 1
device pointing down: pitch = 1

I just tried this on Android and it works fine, so this might just be an iOS issue.


Load Hadoop native libraries

This should load the native libraries for faster HDFS access from local HDFS clients.

Closing in favor of a single PR...


Validate input when setting the QuantumCircuit data attribute directly

Summary
This commit adjusts how users can set a QuantumCircuit's data attribute
directly. Accessing and reading it has no changes, but when just directly
changing it's value outside of the circuits API we now will validate the
input to ensure that the value being used conforms to the expectations
and data format used throughout the circuit's api. Prior to this patch it
was possible to assign arbitrary data in the data parameter which would
make it difficult to add performant modifiers and methods in the circuit's
api because we would constantly have to verify the state of the data
attribute. This commit should fix this because now that the data is
always validated when we build out the circuit's api in the future.
Co-Authored-By: Kevin Krsulich<EMAIL_ADDRESS>Details and comments
Fixes #2826

Added a few commits to make QuantumCircuit.data a list-like MutableSequence wrapper, with the goal being that existing operations on circuit.data should continue to work, while maintaining a consistent state in circuit._data.

It looks like tests are failing because we have to update the pickle files in the mapper tests. I hit this before when I first added the _data attribute. (I'm still not sure why we need to save objects as a pickle instead of just generating things in a setup or setupclass method)


feat: migrate phpunit config-file to version 9

Migrated vie the default --migration parameter.
coverage is added by default.

Thanks!


hello can you make the reader of webrtc html , css dynamic

Describe the feature
instead of hard coded in the repository itself , is it possible to provide custom html and css to override the current ui ?

There's nothing stopping you from making a completely custom html frontend that just connects a video element to the webrtc endpoint (using the javascript from the reader page as a guide)
Or do you explicitly want to restyle the page that opens when you go to a MediaMTX stream's webrtc webpage, without hosting anything else?

yes i want to use the base image for mediamtx , then use the ui ( html,css,js) as folder or link

duplicate of #1229


Include path configurations

If I am locating some libraries code outside de default project path, is it possible to specify another directory as STMCubeIde allows?

I am not quite sure what you mean. If you have included files from other locations in the project, then as long as it builds on your local machine, the action will work.

The main point is that i reference files without including the hole path, for example I do
#include <core_mqtt.h> and this file is not located on the build path, is on another folder outside the core, inc paths

That's fine. That's what most people will do. No one likes writing the whole include path for every include. That's what the include path configuration you showed is for.
All the configurations, inclusive the include paths, are saved in the project file and the build action will use them. You will only have to make sure your local directory structure is the same as when the action is building in the cloud.
Are all project files in the same repo in github? What is the actual error you are getting? It can't find core_mqtt.h? Add find in your pipeline, so you can see a listing of all files that are there.

Were you able to get this to work or are you still having problems?

I'll just assume you got it working and close the issue.
Always a bit disappointing when people are happy to take the time to report an issue they are having, but don't report back if the free help has resolved their problem.

Were you able to get this to work or are you still having problems?

Working, it was my configurarion

working approach https://github.com/smartnexus/slope-monitoring-freertos

thanks for your help, it was my fault sorry :(

Good to hear you have got it working now. Thanks for the update.


Make StrongPath Hasheable

https://hackage.haskell.org/package/hashable-<IP_ADDRESS>/docs/Data-Hashable.html#t:Hashable
Ideally we could just forward to the Hasheable instance of Path, but there is some extra info we keep next to the Path, that might not be viable, and we will have to do it on our own.
This is important to enable usage of StrongPath in certain situations, for example as a key in a HashMap.

Actually I am being silly, we should be able to just call toFilePath on it and then hash that! That should be super easy.

This looks fixed: https://github.com/wasp-lang/strong-path/blob/main/src/StrongPath/Instances.hs#L19

Oh right, thanks!


How many times should a function without arguments be tested by default?

When testing a function that doesn't take input arguments, e.g.,
export function f(): void {
   return;
}

By default it would test it Max number of tests times, which is a bit unexpected since the input only has one possible value. I guess we can say that if a function without arguments is worth testing then it must be nondeterministic, although I feel that should be a separate option users can choose since other functions can be nondeterministic too? E.g., having a "nondeterministic" checkbox that users can toggle to enable testing duplicated inputs and have it on by default for any function without input.

Agree that this can be kind of surprising, especially because we state that NaNofuzz lacks support for non-deterministic PUTs.
The backstory for this behavior revolves around the buggy example program random.ts, which is non-deterministic. We used this program in some early pilots but never in an actual study. In very early versions of NaNofuzz, it could test the same inputs multiple times: it didn't keep track of the prior inputs tests, which was understandably confusing to some users in early pilots.
The logic that contains this exception is here: https://github.com/nanofuzz/nanofuzz/blob/6474eb5255494b61729e3add9ce784b2afbe6d40/src/fuzzer/Fuzzer.ts#L206
How to support testing of non-deterministic PUTs in a usable way is worth exploring! Initial thoughts are maybe to pursue a hybrid approach where the tool will run some duplicate tests to detect non-determinism in case it's present but the user is unaware but also allow the user to assert via an option that the program is, might be, or is not deterministic. Then, once non-determinism is detected or asserted, it's a good question what to do next to help the user test the program.
Maybe once we know a program is non-deterministic, either via an automatic check or a manual option, we turn off the input dupe check and, within the results grid, group and summarize same-input test examples together?
It seems like our human/example-based oracle should also allow assertion of multiple expected outputs to support non-deterministic programs. Supporting multiple values/ranges/more sophistication in the human validator's expected output is something we should do anyway and independently of when we add support for non-determinism.
Would love to hear your thoughts on this too!


Fix form validation for phone number.

I don't know the required format and it's not accepting what should be the default format.

Please assign me, I can fix it

Please assign me, I can fix it


Cannot Install on OS X

$ easy_install .
Processing .
Running setup.py -q bdist_egg --dist-dir /Users/ysimonson/Downloads/traviscline-gevent-zeromq-ead6ce8/egg-dist-tmp-K4ASPZ
error: SandboxViolation: chmod('/Users/ysimonson/.python-eggs/gevent-0.13.6-py2.7-macosx-10.6-intel.egg-tmp/gevent/tmpCuNIY3.$extract', 493) {}
The package setup script has attempted to modify files on your system
that are not within the EasyInstall build area, and has been aborted.
This package cannot be safely installed by EasyInstall, and may not
support alternate installation locations even if you run its setup
script by hand.  Please inform the package's author and the EasyInstall
maintainers to find out if a fix or workaround is available.

@jsg2010 can you assist in replicating this? I think it may be related to somewhat cython/pyrex changes in distribute's setuptools clone.
Specifically, does it occur if you use setuptools in a fresh venv vs distribute?
Thanks!

It occurs on a fresh MacOS Lion install, with a fresh mac-ports installed.
I downloaded the dmg (https://distfiles.macports.org/MacPorts/MacPorts-2.0.3-10.7-Lion.dmg) and installed it using the package installer.
then ran these commands to get what i needed:
sudo port selfupdate
sudo port -Rucp upgrade outdated
sudo port install python27 py27-zmq py27-gevent py27-sphinx py27-lxml py27-html5lib py27-anyjson
sudo port select --set python python27
sudo easy_install-2.7 ZODB3 authkit selector whoosh pycurl cython pyhamcrest pyquery zc.queue
sudo easy_install-2.7 gevent-zeromq <——FAILED
sudo port install py27-pip
sudo pip-2.7 install gevent_zeromq <——FAILED
——checked out the project on GitHub:
python setup.py build
sudo python setup.py install    <——Success

I've encountered the same issue.

Last good version of distribute appears to be distribute==0.6.14

I narrowed this down to a change in distribute but didn't end up forming/submitting a patch for it. Bisecting the changesets between 0.6.14 and 0.6.15 would find the culprit. Any takers? I can take another dive soon unless someone else does.


Map costs

Query hive cluster to get mapping of AWS Accounts to Account and AccountClaim CRs.
jira: OSD-10006
Support querying root OUs
jira: OSD-9906

/retest

/lgtm

Closing in favor of continuing development in #219


Applying gravity in XML does not effect android preview

<com.sothree.slidinguppanel.SlidingUpPanelLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/slidingUpPanelLayout"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:layout_gravity="bottom"
    android:gravity="bottom"
    app:layout_gravity="bottom">

None of these gravities do anything, I expect that the UI change accordingly

Looking into code, the SlidingUpPanelLayout.java does not have any conditions to accept the gravity key, Any idea on how?

To be honest, without investigation: no
But I would love to approve a pull request which solves this !


Detected serve event not following specifications (lab 2)

i ran into this compilation error:
Detected serve event not following specifications.
Grading terminated.
have already ensured to upload all files, have getter functions and toString() methods for all classes already. Why am I still encountering such an error?

Hi! I think the responses in #24 might give you some clarity :")

Hi Zandra, have you managed to solve your issue?

Hello! I was getting this for the Detected arrival event. It seems like it went away after I changed the toString() methods for my Server and Customer classes from a long descriptive String (that included some other attributes) to simply their names.
I'm not 100% sure if this is what fixed it because I made a few changes at once. However, I undid everything else (except the toString() method) and tested it, and it was accepted on CodeCrunch.
I have no idea why it worked, but I hope it helps!

Hi, I have managed to resolve the issue by tweeking the toString() method of my Server class. Thanks @axmszr

Hi, I have managed to resolve the issue by tweeking the toString() method of my Server class. Thanks @axmszr

hi merson, may i know what you tweaked about the toString() method? did u reference all parameters of your Server object or there's no need to?

Hi, I have managed to resolve the issue by tweeking the toString() method of my Server class. Thanks @axmszr

hi merson, may i know what you tweaked about the toString() method? did u reference all parameters of your Server object or there's no need to?

hi zandra, I removed all the additional descriptions and simply returned this.name.
I hope this helps!


Mixing bgfx calls and webgl calls

Hello,
This is more a question than an issue but ...
I am trying to grab webcam frames directly on GPU to relax the garbage collector.
This is trivial using raw JS/WebGL, we just need to pass the HTML5 video element as pixel source in the texImage2D call. Frome webgl 1.0 spec :
typedef (ImageData or HTMLImageElement or HTMLCanvasElement or HTMLVideoElement) TexImageSource;

void texImage2D(GLenum target, GLint level, GLenum internalformat,
                    GLenum format, GLenum type, TexImageSource? source); // May throw DOMException

and some examples can be found around the net.
Using Emscripten, there is no-way to get an html5 video element in c++ side and send it to bgfx to update a texture, therefore it must be done into an embedded JS library.
I have managed to retrieve the WebGLRenderingContext, created by bgfx, in JS side but ....

will it be problematic to update the texture in JS side using WebGLRenderingContext.texImage2D ?    As bgfx in asm.js mode is single-threaded, i imagine it is OK and will not conflict with bgfx command buffer, right ?

Sorry for the issue-question :)
Regards,
Harold

After further reflexion, mixing bgfx and raw WebGL calls should be a really bad idea.
Btw, supporting texture update from video element on AsmJS platform should be a real plus for people working with html5 videos and bgfx :)
But before that, Emscripten has to support transfering html5 elements to c++ side, i will see with them and close subject here.
Regards,
Harold


Fixed error about Rx/Tx coil

Thanks so much!
Tha't something I was not 100% sure about, glad it's clarified.

Also, please note that the book is currently in a "live" state, with a single "latest" version, which is pushed manually. It is the final exam of my class next week, so I won't push updates manually at this time. But these changes will be reflected on the website next time I push a fresh compilation.


Independant spinbox arrow step precision

The arrow buttons on Spinbox become relatively useless when one wants to have the ability to write numbers with high precision. For example setting step to 0.000001. In its current form the arrow buttons will become relatively useless.
This pull request fixes that by implementing an independent step value for the arrow buttons on spinboxes. Default value is 0 to avoid breaking compatibility, and if range class step value is higher it will use that step instead.

This is related to my proposal https://github.com/godotengine/godot-proposals/issues/226
This work has been kindly sponsored by IMVU.

I will likely find a few uses for this in the Godot editor itself :slightly_smiling_face:

@johannesgunnar Is this still desired? If so, it needs to be rebased on the latest master branch.

Thanks for the heads up @aaronfranke. I will take a look at this.

@johannesgunnar Is this still desired? If so, it needs to be rebased on the latest master branch (again). The previous time you rebased, this PR did not pass the style checks. You can fix this by running misc/scripts/clang_format.sh.

@johannesgunnar Is this still desired? If so, it needs to be rebased on the latest master branch (again). The previous time you rebased, this PR did not pass the style checks. You can fix this by running misc/scripts/clang_format.sh.

Ah, thanks for the ping @aaronfranke. Will take a look at this.

Ah, thanks for the ping @aaronfranke. Will take a look at this.

Terribly sorry about the lack of work on this. I have been unable to do any work in the past few months because of health issues and am currently in occupational therapy and pretty much avoiding computers as much as possible. If this feature is desired by someone then this would be a nice junior job for someone that wants to dip their toes into Godot development.

I think it would be better to re-implement those buttons as being side-by-side, instead of two really small ones on top of each other.

What's needed to merge this? I can bring it up to date it if necessary.

@EIREXE Someone needs to rebase this and address all of the above feedback. If you do this, then you can submit a new pull request.

Superseded by #54792.


Add Srikruthi1's file

created a branch, created a file and made a commit, and opened a pull request


chore: add .DS_Store to .gitignore

https://en.wikipedia.org/wiki/.DS_Store

Thanks for this PR, but I think .DS_Store should be configured by Mac developers in global .gitignore rather than in project space.


Failing expect build in Safari and IE

I'm not 100% sure this is an is-equal bug, but it seems like it.
The expect build is failing in Safari and IE. See https://travis-ci.org/mjackson/expect/builds/96101892
It looks like the stack traces originate from inside is-equal. Would you mind taking a look, @ljharb?

Sure - I'll look into this today.
https://github.com/mjackson/expect/blob/master/modules/__tests__/toEqual-test.js#L32-L35 appears to be one of the failing tests.
I suspect this is because Map/Set in Safari 8 can only be iterated by for..of - I'll probably have to use a technique like https://github.com/ljharb/map-tojson/blob/master/index.js#L9-L28 for that, which is pretty ugly.
IE 9 - 11 I'm not sure, but I'll look into that today also.

OK so. In Safari 8, Symbol.iterator does not exist - Map and Set do, but the only way they are iterable is with for..of or their forEach method. I'll have to add some fallback code to handle that case.

Hmm, I wonder if that's the case in IE as well.

IE 11 seems to pass with my Safari 8 fix. Pushing up soon.

I am seeing a breakage in the redux build (which uses expect). It seems to be due to object key order.
var isEqual = require('is-equal')

var a = {a: 1, b: 2, c: 3}
var b = {a: 1, c: 3, b: 2}

console.log(isEqual(a,b))  // prints false

Ah, sorry @ellbee. I may have made the switch to is-equal prematurely. Can you please open an issue on expect and reference this one?

No worries, will do.

Also please file a separate issue here - I think https://github.com/ljharb/is-equal/commit/2517c2151d57451f7f7009df065bf1601316ee46 broke this, by causing object key ordering to start mattering. I'll fix that ASAP also.

@ellbee @mjackson the ordering thing was an unintentional breaking change; I've reverted that and v1.3.1 will be published that fixes that (as well as this issue)

Great, thank you.

:tada: Thanks for the quick turnaround here, @ljharb! I've updated expect to use is-equal 1.4.0. Here's to a bright future making testing better :)

\( ﾟ◡ﾟ)/ please file any issues that come up and i'll get to them asap :-)


[test] Flutter Widget mock should have nullable Key

Flutter now has nullable Key
https://github.com/flutter/flutter/blob/338ade81daf035827294f2f08f68cd3b6b32086f/packages/flutter/lib/src/widgets/framework.dart#L322
however, in the mock it is not nullable
https://github.com/dart-lang/sdk/blob/f0f94705a49fca9789490987bfc5181d71a3b827/pkg/analysis_server/test/mock_packages/flutter/lib/src/widgets/framework.dart#L51

https://dart-review.googlesource.com/c/sdk/+/210128

Could we have a review for the CL please? Merge conflict has been resolved.

I apologize that this was missed for such a long time. The CL looks fine and it's running though the bots now.


Request-Transformer Plugin does not replace headers when append.headers is set

Summary
We are running kong 0.10.3 with the request-transformer plugin to add user-ids and some other values to our incoming requests.
Authentication is handled through key-auth and we have global plugins for each consumer that have the relevant ids:
        {
            "config": {
                "add": {
                    "body": {},
                    "headers": {},
                    "querystring": {}
                },
                "append": {
                    "body": {},
                    "headers": [
                        "X-User-Id:2ef463b1-6a55-4ca3-b265-cf72c5da94db",
                    ],
                    "querystring": {}
                },
                "remove": {
                    "body": {},
                    "headers": {},
                    "querystring": {}
                },
                "rename": {
                    "body": {},
                    "headers": {},
                    "querystring": {}
                },
                "replace": {
                    "body": {},
                    "headers": {},
                    "querystring": {}
                }
            },
            "consumer_id": "904a1380-03cd-4ef4-859c-7ddebb704dfb",
            "created_at":<PHONE_NUMBER>313,
            "enabled": true,
            "id": "08e330a5-3057-477d-b396-f0ea879d74e0",
            "name": "request-transformer"
        }

If I omit the X-User-Id parameter the correct values are set on the upstream request.
If the user sends the X-User-Id header the value is not correctly replaced but just upstreamed directly.
Steps to reproduce:
http http://service/ X-Api-Key:foo => X-User-Id header = foo
http http://service/ X-Api-Key:foo X-User-Id:bar => X-User-Id header = bar and X-Api-Key = foo
This is contrary to the docs where it clearly states:

List of headername:value pairs. If the header is not set, set it with the given value. If it is already set, a new header with the same name and the new value will be set.

The current behavior obviously allows impersonation attacks since any user knowing the user-id of another user can trick the upstream services into thinking they are actually someone else.

Have you tried the replace option? If you add both, it will add it if non-existing, or replace it if it does exist

@Tigraine If you use append, all the values would set to same header separated by comma.  Like following example. So check if upstream server getting comma separated value for X-User-Id
curl "http://mockbin.com/request?name=foo&name=bar" -H "x-header:foo" -H "x-header:bar"
{
  "startedDateTime": "2017-07-25T20:01:06.135Z",
  "clientIPAddress": "<IP_ADDRESS>",
  "method": "GET",
  "url": "http://mockbin.com/request?name=foo&name=bar",
  "httpVersion": "HTTP/1.1",
  "cookies": {},
  "headers": {
    "host": "mockbin.com",
    "connection": "close",
    "accept-encoding": "gzip",
    "x-forwarded-for": "<IP_ADDRESS>, <IP_ADDRESS>",
    "cf-ray": "3841b58cd79c262f-DFW",
    "x-forwarded-proto": "http",
    "cf-visitor": "{\"scheme\":\"http\"}",
    "user-agent": "curl/7.51.0",
    "accept": "*/*",
    "x-header": "foo, bar",
    "cf-connecting-ip": "<IP_ADDRESS>",
    "x-request-id": "c4c26d6b-4feb-401c-9587-dc05acdbe209",
    "x-forwarded-port": "80",
    "via": "1.1 vegur",
    "connect-time": "6",
    "x-request-start": "1501012866125",
    "total-route-time": "0"
  },
  "queryString": {
    "name": [
      "foo",
      "bar"
    ]
  },
  "postData": {
    "mimeType": "application/octet-stream",
    "text": "",
    "params": []
  },
  "headersSize": 505,
  "bodySize": 0
}

@Tieske I'll try that as a workaround - is this now intended behavior or not?
@shashiranjan84 If you read the documentation closely it clearly states that it will replace the current value: If it is already set, a new header with the same name and **the new value** will be set This is either a mistake in the documentation and we have been using this incorrectly so far or a bug in the plugin

List of headername:value pairs. If the header is not set, set it with the given value. If it is already set, a new header with the same name and the new value will be set.
I don't see problem with documentation, it basically means a new header with same name and new value will be added to the request without removing the old. I do not see any mention of replacing existing header value. But we will update the doc to make it more explicit.
Logic works as expected, here is the test
https://github.com/Mashape/kong/blob/master/spec/03-plugins/15-request-transformer/02-access_spec.lua#L1015
Closing this card.

Hmm. Ok - then it works. Although this could be clearer in the docs.

@Tigraine docs are here https://github.com/Mashape/getkong.org/tree/master/app/plugins , a PR that clarifies this is most welcome :+1:

Done - thanks for the awesome work :)


root development

14.adventitious root development ISA root development
GO:0048364
Def: The process whose specific outcome is the
progression of adventitious roots over time, from its
formation to the mature structure.Adventitious roots
are post-embryonic roots that develop from the plant
stem.
Ref: TAIR:tb
15.root cap development ISA root development
GO:0048364
Def: Process involved in the development of a root
cap, which is made of a group of cells that covers
the apical meristem of a root.
Ref: PMID:16006581
Reported by: jenclark
Original Ticket: "geneontology/ontology-requests/3223":https://sourceforge.net/p/geneontology/ontology-requests/3223

thanks www.movied.org


Paper details page should list the authors

Scenario
Given a paper with an author
When a user visits the paper's show page
Then it should show the author's full name
Error

Expected to find text "alan turing" in "title: computing machinery and intelligence\nvenue: mind 49: 433-460\nyear: 1950\nedit this paper | back to papers\ndestroy this paper"

39 exercise tests have passed. There are 44 in total. You will solve multiple at once towards the end.

Sorry, it's not working for me yet:
Scenario
Given a paper with an author
When a user visits the paper's show page
Then it should show the author's full name
Error

Expected to find text "alan turing" in "title: computing machinery and intelligence\nvenue: mind 49: 433-460\nyear: 1950\nauthors:\nedit this paper | back to papers\ndestroy this paper"

If you have problems solving this task, please don't hesitate to contact the teaching team!

Sorry, it's not working for me yet:
Scenario
Given a paper with an author
When a user visits the paper's show page
Then it should show the author's full name
Error

Expected to find text "alan turing" in "title: computing machinery and intelligence\nvenue: mind 49: 433-460\nyear: 1950\nauthors:\nedit this paper | back to papers\ndestroy this paper"

If you have problems solving this task, please don't hesitate to contact the teaching team!

Sorry, it's not working for me yet:
Scenario
Given a paper with an author
When a user visits the paper's show page
Then it should show the author's full name
Error

Expected to find text "alan turing" in "title: computing machinery and intelligence\nvenue: mind 49: 433-460\nyear: 1950\nauthors:\n(maybe even alan turing)\nedit this paper | back to papers\ndestroy this paper". (however, it was found 1 time using a case insensitive search.)

If you have problems solving this task, please don't hesitate to contact the teaching team!

naja...


Pulling all Secrets from Provider

This is just a question more than anything else, apologies if it's simple and I just haven't been able to find the answer.
Having tested pulling a single 'ExternalSecret', I would now like to be able to pull all secrets present within AWS secrets manager. I'm currently doing this for the single secret (in Terraform I might add):
resource "kubectl_manifest" "external_secrets" {
  yaml_body  = <<-EOF
    apiVersion: external-secrets.io/v1beta1
    kind: ExternalSecret
    metadata:
      name: eks-external-secret
    spec:
      refreshInterval: 1m
      secretStoreRef:
        name: cluster-store
        kind: ClusterSecretStore
      target:
        name: test
      data:
      - secretKey: testing-first-secret
        remoteRef:
          key: test
          property: test
          EOF

However, now that I've managed to do this successfully, I need to try and pull hundreds of secrets from secret manager. Is there a built in function that would allow me to do this, rather than basically duplicating what I've done above hundreds of times?
Cheers!

hey! You can use find by name/regexp or tags:
https://external-secrets.io/v0.7.2/guides/getallsecrets/#fetching-secrets-matching-a-given-name-pattern

hey! You can use find by name/regexp or tags:
https://external-secrets.io/v0.7.2/guides/getallsecrets/#fetching-secrets-matching-a-given-name-pattern

Hey moolen, thanks for that, that's pretty much exactly what I was looking for 😄
This does pose another question though having just tested this; this is an example of a test I just ran:
kubectl get secret kubernetes-tagged-secrets -o yaml
apiVersion: v1
data:
  test: eyJ0ZXN0IjoidGVzdCIsInRlc3R0IjoidGVzdHQiLCJ0ZXN0dHQiOiJ0ZXN0dHQiLCJ0ZXN0dHR0IjoidGVzdHR0dCJ9
  test2: eyJ0ZXN0MiI6InRlc3QyIn0=
  test3: eyJ0ZXN0MyI6InRlc3QzIn0=
  test4: eyJ0ZXN0NCI6InRlc3Q0In0=
  test5: eyJ0ZXN0NSI6InRlc3Q1In0=
  test6: eyJ0ZXN0NiI6InRlc3Q2In0=
  test7: eyJ0ZXN0NyI6InRlc3Q3In0=
  test8: eyJ0ZXN0OCI6InRlc3Q4In0=
  test9: eyJ0ZXN0OSI6InRlc3Q5In0=

This is a good first step, however, I'd like to get them a bit better organised that this. For starters, under 'test', by adding additional key/values to the secret in question, it simply appends the additional keys and values in a list format rather than having them organised line-by-line, which would be more ideal to grab from an application perspective (and just neater in general).
Do you have any pointers as to how to best go about achieving this? I'm conscious this is kinda another issue in itself but it may be useful for future users to have both answers in one place.
Cheers!

Hey, you can take a look at this:
https://external-secrets.io/v0.7.2/guides/datafrom-rewrite/
https://external-secrets.io/v0.7.2/guides/templating/
How do you want the resulting secret to look like?

Hey, you can take a look at this: https://external-secrets.io/v0.7.2/guides/datafrom-rewrite/
https://external-secrets.io/v0.7.2/guides/templating/
How do you want the resulting secret to look like?

That looks to be in the right direction, thank you!
How I imagine the secrets to look is something like the following (using the output from my previous response as reference):
test:
  test: eyJ0ZXN0Ijo
  testt: joidGV
  testtt: c3R0Ij
  testttt: GVzdHQiL

So I guess basically, it would be good to have the keys for each of the secret entries being non-encoded (as those aren't the sensitive values) but that's not a huge deal if that's not possible to do, but the way in which this is formatted is much more how I imagined it playing out having reviewed the documentation you send previously.
We could make it work how it is today, but it would require a significant per-application refactor. If you have any suggestions let me know, otherwise I'll start experimenting and see what I can come up with.
Thanks for all your help so far by the way!

Hey @hjeffries94 looking at your example test=eyJ0ZXN0... it looks like you want to extract information from structured json data. You can do that with dataFrom[].extract.
Given a secret foo/bar/baz with value:
{
"test":"test",
"testt":"testt",
"testtt":"testtt",
"testttt":"testttt"
}

You can extract the information of this single secret
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: example
spec:
  # ...
  dataFrom:
  - extract:
      key: foo/bar/baz

That should give you the secret:
kind: Secret
# ...
data:
  test: test
  testt: testt
  testtt: testtt
  testttt: testttt

Note:

:warning: you need to know the secret name in advance, you can not use both dataFrom[].find and dataFrom[].extract
:warning: when specifying multiple dataFrom[].find you may run into a key conflict, please take a look at this guide.

Your snippet above suggests you want to use nested data test.test, test.testt ... in a Kind=Secret - this is not possible, you can only define key/value pairs, see docs.
test:
  test: eyJ0ZXN0Ijo
  testt: joidGV
  testtt: c3R0Ij
  testttt: GVzdHQiL


[QuickAccent] Shift + Space to navigate backwards

Summary of the Pull Request
Detect shift key to navigate characters backwards with activation key set to space.

PR Checklist

[x] Closes: #21548
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Set activation key to space (also test with left/right/space)
Activate quick accent
Validate that shift + space is navigating characters backwards (left/right shift)

Hi @davidegiacometti ,
Thanks for opening this PR.
This change was discussed before here: https://github.com/microsoft/PowerToys/pull/20320#issuecomment-1253708634
Did you take this into account? How's the interaction when trying to do uppercase accented letters?

@jaimecbernardo I have missed that!
I have tested with uppercase (SHIFT) and when the selector appear if you dont release SHIFT it's navigating backwards.
A better key could be CTRL (ALT is very inconvenient) or we can decide to close the issue as wont-fix.
The author of the issue got a valid point:

if you mistakenly press space one time too many and just want to back to the previous character instead of going through the whole list.

I like this idea!
But something disturb me. I stroke shift + E + space to get the uppercase selector.
And naturally, I keep shift + E pushed, and repeat space. But it go backwards, it's troubling, so I have to up shift key, before changing character. It's not a big deal, but can be disturbing for some.
So I propose to add an option (activated by default), to deactivate it if some people get too much trouble with this feature.

The other solution I see, is to let forward, and activate backward if the shift key, has been up and down again. But may be a bit more complicated to do.

I have changed the code to detect shift for navigating backwards only if shift if pressed with visible toolbar.
If you press SHIFT + A + SPACE is navigating forwards. To navigate backwards you need to release and press SHIFT again.
Let me know what do you think and if this needs to be configurable.

@jaimecbernardo any reason to not merge?

Nice catch @taras-janea
I have fixed conflicts with main and the left/right shift behavior.

Looks good and works well 👍
@davidegiacometti thank you for your contribution!


Feature: Passing Builds

Reopened because Codecov is failing again.

It appears to have been a momentary interruption in service. Things are working again. :+1:


Camel 8973

Adding transacted batching consumer component to camel-sjms
https://issues.apache.org/jira/browse/CAMEL-8973

Thanks for this PR the code has been merged do you mind closing it?


Integrate with bloc-did-method for sidetree-mock

@fqutishat we need to show the static content feature hosted by the peers. is that a different issue?

@fqutishat we need to show the static content feature hosted by the peers. is that a different issue?

yes I have different ticket for that https://github.com/trustbloc/edge-sandbox/pull/137


Crash in AppStateCollector.getIpAddresses

I am using your library (com.siftscience:sift-android:0.10.4), and am seeing this crash (so far only for one of my users):
Fatal Exception: java.lang.NullPointerException: Attempt to invoke interface method 'boolean java.util.Enumeration.hasMoreElements()' on a null object reference
at siftscience.android.AppStateCollector.getIpAddresses(AppStateCollector.java:172)
at siftscience.android.AppStateCollector.get(AppStateCollector.java:150)
at siftscience.android.AppStateCollector.doCollect(AppStateCollector.java:127)
at siftscience.android.AppStateCollector.collect(AppStateCollector.java:90)
at siftscience.android.Sift$1.run(Sift.java:106)
at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
Looking at your code, AppStateCollector.getIpAddresses has this:
for (Enumeration en = NetworkInterface.getNetworkInterfaces();
en.hasMoreElements(); ) {
However, in Android library 28 the function getNetworkInterfaces sometimes returns null. It actually has a comment explaining the null return:
// Android-changed: Rewrote NetworkInterface on top of Libcore.io.
// specified to return null if no network interfaces
Thus you need to make a minor modification to getIpAddresses so you check for null before starting the for loop.
Enumeration en = NetworkInterface.getNetworkInterfaces();
if (en != null) {
while (en.hasMoreElements) { ... }

Thanks Bruce - acknowledging the request. Tagging in David and Gary to take
a look.
On Fri, May 3, 2019 at 1:18 PM Bruce Clift<EMAIL_ADDRESS>wrote:

I am using your library (com.siftscience:sift-android:0.10.4), and am
seeing this crash (so far only for one of my users):
Fatal Exception: java.lang.NullPointerException: Attempt to invoke
interface method 'boolean java.util.Enumeration.hasMoreElements()' on a
null object reference
at
siftscience.android.AppStateCollector.getIpAddresses(AppStateCollector.java:172)
at siftscience.android.AppStateCollector.get(AppStateCollector.java:150)
at
siftscience.android.AppStateCollector.doCollect(AppStateCollector.java:127)
at siftscience.android.AppStateCollector.collect(AppStateCollector.java:90)
at siftscience.android.Sift$1.run(Sift.java:106)
at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
Looking at your code, AppStateCollector.getIpAddresses has this:
for (Enumeration en = NetworkInterface.getNetworkInterfaces();
en.hasMoreElements(); ) {
However, in Android library 28 the function getNetworkInterfaces sometimes
returns null. It actually has a comment explaining the null return:
// Android-changed: Rewrote NetworkInterface on top of Libcore.io.
// specified to return null if no network interfaces
Thus you need to make a minor modification to getIpAddresses so you check
for null before starting the for loop.
Enumeration en = NetworkInterface.getNetworkInterfaces();
if (en != null) {
while (en.hasMoreElements) { ... }
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SiftScience/sift-android/issues/67, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABNDSOHTELD5USWUR5MJGZ3PTSMYPANCNFSM4HKWWVSA
.

Update: this is fixed in the latest version, @garylee1 will update Github with the latest release shortly and we'll close this out.

The fix has been released to both Maven and Github – https://github.com/SiftScience/sift-android/releases/tag/v0.10.5
Thank for your patience, and for your surfacing this!


Create ClientSession inside coroutine to avoid warnings

The ClientSession is now reused across the life of the instance. It can be closed and fixes warnings at startup


Incorrect Module setting text and help

On the module settings we have an option with the following text and help:
Text: Hide Admin Border
Help: Check this box to hide the red border that shows around the module when it is only visible to Administrators.
In DNN 9 there's no such thing as a red border.
Instead the system displays a message notification on top of the module with the text "Visible by Administrators only."
I suggest changing the text and help to:
Text: Hide Admin Only Visibility Message:
Help: Check this box to hide the message notification that is displayed on top of the module when it is only visible to Administrators.

This has been reworded as "Admin message" instead of "Admin border" somewhere between the time this issue was open and Dnn 9.6.2 was out. Closing this issues.


Describe best practices for provisioning resources

Feature Suggestion
Currently there is no description of how to architect a provisioning model for plugins and the resources they provision.
Possible Implementation
Documentation describing the best practices and architecture, coupled with visual aids.
Context
This will help building new plugins to provision things from scratch.

Given that there are no best practices form Backstage on this topic I will close this issue and re-open it later in the ServiceBroker context where it belongs.


New georeference modal base

First step to finish #4158

Implements the base for the new modal. I followed more or less the same pattern as for the merge modal that worked great there, i.e.

one model/view couple per geocoder option.
a parent view (georeference_view) that handles the life-cycle and high-level changes of the view, details are delegated to the currently selected geocoder option.

All the tabs are there as can be seen in the screenshot but the others are just stub models, and will be implemented separately, as outlined in the referenced issue above.

@xavijam @javierarce please have a look at the code.
I'm getting more and more troubled by how the CSS is organized, I think we would benefit from having a discussion at some point soon on each of our experiences following the new styleguide and how to organize things for reusability and make things a little more consistent.
cc @saleiva + @josecruz for the UI

Frontend tests were OK :+1: (details)

Yeah, I share your concerns about the stylesheet organisation :|

Agree, next week is a good moment

:+1: code looks good to me.


First e2e pass

Perform the first e2e pass on docs writing:

Overview: Page with brief explanation on the protocol and why it's useful. Index to other section
Get Started: General concepts, more detailed than overview, not going into technical details. Can cover concepts not invented by us (eg. cover CRDTs here for protocol docs)
Technical Overview: Broad technical overview
Tutorials: How to build with the protocol (eg. build grid from scratch with cursor)

whole doc restructured. this issue is outdated


Get the postscript name of a font

There is currently no way to get the postscript name of a font (i.e. "Roboto-Medium"). This is often very useful for identifying fonts. Is this currently planned?

https://github.com/dylanede/stb_truetype-rs/pull/4
I've written a pull request for stb_truetype-rs (backend library of rusttype) but it is not merged.
I believe this PR is useful if someone implement font-name-related feature to rusttype.

@lo48576 I somehow have the feeling this repository is not maintained anymore...


diff_match_patch is not a constructor when using Require.js

I have a project using Require.js (ver 2.1.18) where I'm using CodeMirror (ver 5.5.1) to view/edit a JavaScript file in the browser (this and all CodeMirror addons work as expected).
I now need to show any diffs in the file, so I've been trying to implement the merge.js addon, but having troubles with the diff_match_patch.js lib that is also required.
// Firebug + Firefox 39.0.3
// merge.js
// (line 550, col 12)
"TypeError: diff_match_patch is not a constructor
var dmp = new diff_match_patch();"

// Chrome Version 44.0.2403.130 (64-bit)
// merge.js (551)
"Uncaught TypeError: diff_match_patch is not a function"

I can see that diff_match_patch.js is being loaded by Require, but when passed to that function in merge.js, the JavaScript error breaks the page. The Merge view doesn't show.
Changing the line
var dmp = new diff_match_patch();

to
var dmp = diff_match_patch;

allows the 2 views to render, but the dmp.diff_main map is not created.

Unfortunately, diff_match_patch isn't packaged very well -- it doesn't know that there is such a thing as module loaders, and just assumes that it is okay to dump four variables into global scope.
I managed to get it working with a small change to merge.js (which tried to access diff_match_patch through an interface object). See this gist for the working requirejs-based code.

Thanks @marijnh - working great on my end too!

@marijnh Is that gist still valid? It does not render any view for me.
Also @clintmilner assuming you are using typescript, how did you declare a global variable?
I added
"@types/diff-match-patch": "^1.0.32",
"diff-match-patch": "^1.0.1"

to my package.json but I still get
ERROR ReferenceError: diff_match_patch is not defined
    at getDiff (merge.js:649)
    at DiffView.init (merge.js:52)
    at new CodeMirror.MergeView (merge.js:570)
    at Function.CodeMirror.MergeView (merge.js:536)

@ajinkyasurya I dumped Require.js about a year ago, and I can't find the commit where I had this + Require.js working. I do know that I was wasn't importing CodeMirror-related files vai a package.json file. Perhaps try importing manually?

Hacking the globals with webpack, works for me:
new webpack.ProvidePlugin({
    diff_match_patch: 'diff-match-patch',
    DIFF_EQUAL: ['diff-match-patch', 'DIFF_EQUAL'],
    DIFF_INSERT: ['diff-match-patch', 'DIFF_INSERT'],
    DIFF_DELETE: ['diff-match-patch', 'DIFF_DELETE'],
}),


[iOS] Crash on ImageDecoderImpeller::Decode

Steps to reproduce
Unfortunately, we couldn't reproduce that issue locally.
Expected results
The app to no crash.
Actual results
App crashed.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
          Crashed: io.worker.3
0  Flutter                        0x5750b4 flutter::ImpellerAllocator::allocPixelRef(SkBitmap*) + 583 (image_decoder_impeller.cc:583)
1  Flutter                        0x571148 std::_fl::__function::__func<flutter::ImageDecoderImpeller::Decode(fml::RefPtr<flutter::ImageDescriptor>, unsigned int, unsigned int, std::_fl::function<void (sk_sp<flutter::DlImage>, std::_fl::basic_string<char, std::_fl::char_traits<char>, std::_fl::allocator<char>>)> const&)::$_1, std::_fl::allocator<flutter::ImageDecoderImpeller::Decode(fml::RefPtr<flutter::ImageDescriptor>, unsigned int, unsigned int, std::_fl::function<void (sk_sp<flutter::DlImage>, std::_fl::basic_string<char, std::_fl::char_traits<char>, std::_fl::allocator<char>>)> const&)::$_1>, void ()>::operator()() + 216 (image_decoder_impeller.cc:216)
2  Flutter                        0x2ec5f0 fml::ConcurrentMessageLoopDarwin::ExecuteTask(std::_fl::function<void ()> const&) + 18 (concurrent_message_loop_factory.mm:18)
3  Flutter                        0x2e62d4 void* std::_fl::__thread_proxy[abi:v15000]<std::_fl::tuple<std::_fl::unique_ptr<std::_fl::__thread_struct, std::_fl::default_delete<std::_fl::__thread_struct>>, fml::ConcurrentMessageLoop::ConcurrentMessageLoop(unsigned long)::$_0>>(void*) + 1372 (vector:1372)
4  libsystem_pthread.dylib        0x30ec _pthread_start + 116
5  libsystem_pthread.dylib        0x172c thread_start + 8

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.19.4, on macOS 14.4.1 23E224 darwin-arm64, locale en-AE)
    • Flutter version 3.19.4 on channel stable at /Users/brunoreginato/fvm/versions/3.19.4
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 68bfaea224 (3 months ago), 2024-03-20 15:36:31 -0700
    • Engine revision a5c24f538d
    • Dart version 3.3.2
    • DevTools version 2.31.1

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/brunoreginato/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 75.1.4
    • Dart plugin version 232.10072.19

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.90.0

[✓] Connected device (3 available)
    • iPhone Xs (mobile) • 121672A1-B308-4EAC-8031-0CF1665476C8 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 125.0.6422.142
    ! Error: Browsing on the local area network for Ahmed Henawey’s iPhone. Ensure the device is unlocked and attached with a cable or associated with the
      same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Closing this as duplicate of https://github.com/flutter/flutter/issues/138897


[Event Request] Codeunit 6620 Copy Document Mgt. - function CopySalesDoc

Dear,
I request for below Integration

Definition of Integration as below

thanks & regards
Ashwini Tripathi

Probably you wanted to write "if not IsHandled then"?

Yes, You are correct, my logic wants to skip Copying Sales Line below code.

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.


Android Basics: Get data from the Internet

name: Android Basics Unit 4 issue
about: Report problems with the Android Basics Unit 4 codelabs
title: 'Android Basics Unit 4 issue:'
labels: ''
assignees: ''

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-getting-data-internet?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-4-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-getting-data-internet#4
In which task and step of the codelab can this issue be found?
Add Retrofit dependencies
Open the project's top-level level build.gradle(Project: MarsPhotos) file. Notice the repositories listed under the repositories block. You should see two repositories, google(), mavenCentral().
Describe the problem
Got the following error when adding repositories in the build.gradle file and then syncing the changes:
> Build was configured to prefer settings repositories over project repositories but repository 'Google' was added by build file 'build.gradle'

According to stackoverflow:
"Gradle 6.8 introduced central declaration of repositories, new way to define repositories."
And repositories should be added in settings.gradle if I understand correctly (I'm new to android development).
Versions
Android Studio version:
Android Studio Bumblebee | 2021.1.1 Patch 2
Build #AI-211.7628.21.2111.8193401, built on February 17, 2022
Runtime version: 11.0.11+0-b60-7590822 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 12.3.1
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 16
Registry: external.system.auto.import.disabled=true
Non-Bundled Plugins: com.facebook.tools.metaplugin (1.3.3)

Maybe it's because I didn't clone the starter app provided in the beginning of the tutorial and created my own app manually in Android studio.


2915 2023 incorrect photo

I have accidentally uploaded an incorrect photo for 2915's 2023 robot. The photo in question is the one without bumpers.
https://www.thebluealliance.com/team/2915/2023

Removed.


fix main and types pointer

This fix solves this annoying warning that appears anytime the script runs:

Of course this fix should be also sent to npm to update public package..
Hope it helps!

Hey, thanks for the heads up. It used to be like this but it was causing issue with linkage. I assume the warning is happening due to Node v16. For the moment we will leave it as is.


Add domain validation during ImgixClient initialization

Adds a validation that will throw an error if a user attempts to initialize a new ImgixClient object with an invalid domain.
The validation criteria is based off of the same regex used on dashboard.imgix.com

:tada: This PR is included in version 0.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove Blockchain dependency from GetBlockHeadersMessageHandler and GetBlocksMessageHandler

This dependency is because to process the BlockHeader the access to the repository is need and the methods that access the repository are in the Blockchain class.
Blockchain class should be the maestro of the synchronization and initializing it.
Smaler classes with more specific behavior should be responsible for the retrieve of the data like: BlockModel, TransactionModel
this objects should have the access to IRepository and not the blockchain that has a high-level responsibility.

done!


How to Limit the drag when Swipe opening the slide menu.

I'm probably overlooking something but, is it possible to limit the drag and snap animation of the slide menu

see how the body can be dragged to the far right. i want it to stop at where the text of the menu ends. is it possible. i have searched in the issues, i couldn't find any related ones, or i am not searching for the correct term.
thanks in advance.

Any updates with this??

any update on the same


Bug: Incompatibility of Rust's stdlib

This bug was originally found in #53 . The reason was: stdlib uses a PANIC_COUNT in TLS to ensure no panic while panicking in runtime. But obviously, Coroutines in coio can be migrated between threads, which means that it turns out to cause data race (because compiler still think that we are running in the same thread, so we may access to the other thread's TLS without any synchronization method).
We wanted to solve this PANIC_COUNT partially in https://github.com/rust-lang/rust/pull/33408, but because stdlib relies heavily on TLS (such as println!), it will also causes SIGSEGV randomly:
println!("Before");
Scheduler::sched(); // Switch out
// Well, now, this Coroutine may already been stolen by the other thread
// And then resumed by the other thread
println!("After");

Rust's compiler don't know that we have switched to another thread, so it may inline those TLS calls.
We are looking for a solution for this bug, discussing in here, if you have any idea, please help.

how much it affects real-world applications? is there any way to check if given project is affected by imporper TLS handling?

Unfortunately, I think every existing projects in Rust will be affected. Because nearly every Rust projects uses libstd, and libstd uses TLS for

Panic handling
Stdio

So, for example, if someone's project has println, it is already been affected.
println!("Print something");
// ...
Scheduler::sched();
// ...
println!("Print, but possibly crash");

seems there is no short-term workaround either? I think it should be mentioned in README in both coio-rs and mioco.

It seems that someone is going to add coroutine support directly to LLVM, which means that it is possible to tell LLVM not to inline TLS calls between context swaps.
https://internals.rust-lang.org/t/llvm-coroutines-to-bring-awarness

First of all, thanks for the great work what you put into this library. I played with setjmp/longjmp to have simple generators in Rust, then found ucontext and ultimately this crate. My intention was to have Go like coroutines in Rust.
I tried to compile coio but one if its dependency became outdated and today's Rust compiler won't accept it (owning_ref, maybe).
I read the long issue in the std library that TLS should be eliminated from std to support coroutines. I  don't have the knowledge to add any value to that conversation, but FYI LLVM 4.0 has experimental coroutine support, which may help.
Somebody wrote that TLS is just a hidden extra parameter to functions. According to this link, an extra .tdata segment is created in the executable and each thread will get a different copy from this.
I may have another idea: if a TLS is accessed from a coroutine, then the TLS is not placed into .tdata but into the coroutine's captured environment. What do you think about this approach? Is it feasible?

If Goroutines were in Rust it would be a pure awesomeness

futures-await goes in this direction.

Also: https://github.com/rust-lang/rfcs/pull/2033

@ihrwein Thanks for your suggestion. What you are proposing is "coroutine local variable", which stores data with the Coroutine. But as you can see, it requires Rust's official teams' support, and they don't want to provide any.

I wrote down my thoughts about this here: https://github.com/rust-lang/rfcs/pull/2033#issuecomment-309212006
tl;dr: I see the future of coio etc. a bit pessimistic and fully expect a mediocre solution consisting out of async/await and tokio to be the official standard in the future.

maybe too late to the party, but isn't this a matter of just not inlining TLS access?
Might be a little too late here since rust is headed to standardizing futures, but I wrote an entire linker https://github.com/aep/elfkit which can easily be adapted to support a different TLS strategy.

Yup. I have already figure out. This is nothing to do with the TLS inlining.

PANIC_COUNT issue in #70


HD OSSing - What would someone outside of Home Depot gain from it?

Anyone looking for a React implementation of scroll-driven lists with sticky headers will be interested in using this component.


[Bug Report] can't trigger a current-change event by modifying the currentPage in a size-change event

Element UI version
2.10.1
OS/Browsers version
macos / chrome 75.0.3770.100
Vue version
2.6.10
Reproduction Link
https://codepen.io/don93/pen/ewXNxL?editors=1011
Steps to reproduce

click https://codepen.io/don93/pen/ewXNxL?editors=1011
change page-size to 400
check the console, there is no 'handleCurrentChange' output
comment line 19 in js block, which modify the 'p' value
repeat steps 2&3 again, now you can see the 'handleCurrentChange' output back there

What is Expected?
change 'currentPage' in a size-change event should properly trigger current-page event
What is actually happening?
change 'currentPage' in a size-change event does not trigger current-page event

turns out you just can't trigger current-change event by setting currentPage value. I don't know whether it is an issue


Warning molecule

GitHub Issue
fixes https://github.com/HausDAO/daohaus-monorepo/issues/572

I decided to just add the Field Alert into molecules - didnt feel right in atoms
wondering if this can just be Alert as well as children (button) are optional?

Changes

[X] adding FieldAlert Atom
[X] adding stories for FieldAlert
[X] adding Unlock Token Molecule

Packages Added
N/A
Checks
Before making your PR, please check the following:

[X] Critical lint errors are resolved
[X] App runs locally
[X] App builds locally (run the build command for any impacted package and check for any errors before the PR)

Details

Just noticed the border on the primary field alert is blue. Text shouldn't be white either.


After update not work

I had a 4.5 version. I downloaded and setup a new verson 4.6.
Nothing of payloads is not working. At startup payload in C2Server.py window no connection information is displayed and in ImplantHandler.py window no implant, but in version 4.5 all worked.
In Config.py I changed HostnameIP on that has issued the ifconfig command on my interface.
Why not work?

Closing due to lack of activity, feel free to re-open if the issue is still present.


Correlation event testing in testing framework should not be directional

Seems like we're differentiating between EdgeSpec(A, Correlation, B) and EdgeSpec(B, Correlation, A), but Correlations are UndirectedRelations (cf. taxonomy) so we should allow either to pass just fine.

Good point.  The tests were written before this was an issue.  Since correlation is borrowing a lot of causal rules, we have a cause and effect as far as the rule is concerned.  Is that something that needs to be evaluated for some reason?  E.g., should there be a warning?  Making it undirected probably isn't a very difficult change.

This has been addressed.


All events in standard calendar module are shown in UTC following the update to MM 2.12.0

I found a bug in MagicMirror
Following the update to 2.12.0 all Google Calendar events are shown incorrectly.
Platform: Raspberry Pi 3
Node Version: 10.20.1
MagicMirror Version: 2.12.0
Description:
I use Google calendars configured in MM standards "calendar" module via .ical URLs.
My local timezone is MSK (UTC+3, no daylight savings time).
Following the update to MM version 2.12.0, all events are shown as they start/finish 3 hours earlier. I.e. if I cave an event that starts at "15:00" local time, it is now shown in MM as starting at "12:00". It seems that UTC timezone is used instead of local one.
Configuration:
Skipping this step for now as config.js file contains nothing related to time-zones.

It might have something to do with this PR: https://github.com/MichMich/MagicMirror/commit/6d60baa2d61deb43aed0769b0527b9475f725064#diff-4163d379a03c855907bc93633f16efa0
@rejas: Any idea if this could be the case?

Made some more tests. It looks that the problem is impacting only recurring events, one-time events are shown correctly.

https://github.com/MichMich/MagicMirror/pull/2057

Ok, I've merged #2057, could you check if this fixes the issue by running the develop branch?
git checkout develop && git pull

The given merge makes the situation for me event worse, the difference with real time becomes larger.
But it works fine if I remove the minus sign for the "offset" variable:
// correct data for the current timezone
var offset = -(new Date().getTimezoneOffset() * 60 * 1000);
           ^ remove this one
event.startDate = event.startDate - offset;
event.endDate = event.endDate - offset;

After applying #2071 everything seems to be working fine.

Is there a test for this scenario in the code by the way?

Just wanted to weigh in to confirm:

It was happening with recurring events only
The fix seems to have worked (pulled develop.. looking good).

Thank you!  This is the most crucial thing on my MM at the moment since I use it to track work meetings.

Thanks for your confirmation. I'm closing this issue for now. Let me know if I need to reopen this issue.

you should leave these fixed in develop issues open for people to search against, close them on actual release to master

or think about bugfix releases. We will have discussions about this issue and weather bug from now on until next release (here, in the forum, in discord). And telling people to use develop branch or wait 3 months is no solution. Same thing was the socket.io bug in the release before.
May it is a solution to make one bugfix release 2 weeks after main release (if there are such bugs).

I agree, but Mich didn't to point releases, even tho there is numbering for them


Laboratory Work #1, Vyacheslav Sviridov - 20FPL1

Can I update tests somehow? I don't think that I can pass the s_2_5_raw_metadata_test as it checks the html page content and it can't get it without headers

Can I update tests somehow? I don't think that I can pass the s_2_5_raw_metadata_test as it checks the html page content and it can't get it without headers

Of course!
Tests perform default requests with no extra headers and/or encoding settings, if you need them -- add them. But no more than that :)

Should we leave English words while processing or should we better get rid of them as well?

By the way, can I implement passing a score or a string to the TextProcessingPipeline.run() function so I could choose what kinds of processed text files I'd like to get?

By the way, can I implement passing a score or a string to the TextProcessingPipeline.run() function so I could choose what kinds of processed text files I'd like to get?

Ah, I forgot that in that case the tests most likely won't work

By the way, can I implement passing a score or a string to the TextProcessingPipeline.run() function so I could choose what kinds of processed text files I'd like to get?

Ah, I forgot that in that case the tests most likely won't work

Indeed. However, if by the time you are finished with the given instructions you still feel enthusiastic about enhancing your current pipeline with extra functionality, you are very welcome to do so in a separate module. This kind of work is always appreciated, yet it must not interfere with you completing what is mandatory.

The code execution time was more than 20 minutes, 1 article. It seems like mystem just gets stuck while processing a file


How to ask save_png not to pop up a file dialog when filename is given?

figure.save_png(filename) pops up a file dialog on Chrome. I don't need it because I am already giving it a filename with full path. How do I disable the file dialog?

Closed by accident.

What do you want to behavior to be?
PS: The filename is just a filename hint to the browser, it's not a full path, you cannot save to a particular path on the client if that's what you want.

save to a particular path

Alas, that is what I want. Calls the API with given filename, gets a file written, no file dialog.

If you can somehow give me the Numpy array of the data to save, I can possibly use scikit-image to do that without the front-end limitation.
https://scikit-image.org/docs/dev/api/skimage.io.html#skimage.io.imsave

That is possible with https://ipywebrtc.readthedocs.io/en/latest/ImageRecorder.html but you cannot wait for that in a sync way (e.g. you can do image_data = get_image_data(fig), it needs to go through callbacks, possibly using some wrapper with async/await.

it needs to go through callbacks

Not sure what that means for me on the Python side... Is this something Mario can help with?

Fixed by #1397


Updates working but "progress" gear busy "ad infinitum"

Good day Yahnis.
First of all thanks for the outstanding job. I've followed your instructions and I'm able to download and install my plugin's updates from the wp-update-server.
If I fire up the update process from the "update-core" core page (i.e.: http://myexmaplesite.com/wordpress/wp-admin/update-core.php for example), the update process is flawless. Each phase is displayed while in progress (maintenance mode activation, zip file download, decompression, etc.) and, at the end, a message notifies me of the successful update.
On the other hand, if I do the same, but from the "installed plugins" page (i.e.: http://myexamplesite.com/wordpress/wp-admin/plugins.php) the process apparently freezes.
The page displays correctly an "update available" warning, but after clicking the "update now" button the "progress gear" starts revolving, the "please wait" notice is displayed but nothing more happens.
If i try to abandon the page, Wordpress warns me that an update is in progress and if I forcibly leave the page, this could break my plugin.
Well, actually this doesn't happen. Actually the plugin is updated correctly. In fact if you forcibly leave the page and reload the plugin page you can see that the plugin has been updated successfully.
So the problem is, apparently, that Wordpress is awaiting for some form of notification at the end of the process that the update-server doesn't give.
By the way, I'm using wp 4.2.2.
Thank you again for your help

The "update now" button works via AJAX. My guess would be that while WordPress is installing the update there is some unexpected output (e.g. a PHP notice) that then gets included in the AJAX response. As a result, the script that handles the update button fails to parse the response and gets stuck.
To debug something like that, the first thing I would do is look at the HTTP requests sent by the browser. As you probably already know, most modern browsers include a network monitor of some kind. Examine the request that's sent when you click "update now". Usually it will be a POST request to admin-ajax.php. What does the response look like? Or is there no response and the request just times out?
Normally, the response will be a JSON document similar to this:
{"success":true,"data":{"update":"plugin","plugin":"newsletter\/plugin.php","slug":"newsletter","oldVersion":"Version 3.7.8","newVersion":"Version 3.7.9"}}

Hi, Yahnis. Thanks for your answare.
This is the JSON response I get:
{"success":true,"data":{"update":"plugin","plugin":"CkTellNotifier/plugin.php","slug":"cktellnotifier","oldVersion":"Versione 0.1","newVersion":"Versione 0.99rc1"}}

So, apparently there is nothing wrong. The only thing I'm seeing that looks different is the CapitalCase name of my plugin.

Sorry. I Inadvertently hit the "Close and comment" button instead of the "Comment" one.

That JSON looks correct to me. Is there anything else in the response - stray whitespace, perhaps? Are there any error messages in the JS console?
The capital-cased plugin field shouldn't matter - as far as I can tell, the script that runs AJAX updates (/wp-admin/js/updates.js) doesn't actually use it. The slug is the important bit. WP uses that to find the right plugin table row and replace the "updating" message with "updated!".
Anyway, the next thing I would try is sprinkling some console.log()'s into the wp.updates.updateSuccess method in updates.js to see if it actually receives the response and where things go wrong.

Well, I've used Firefox and Safari to monitor network traffic and both don't have a "raw" view for the JSON response. Both display a neat list of key/values, so that i cannot say if there is something wrong in the actual JSON formatting. But my guess is that if the browser is displaying the answer in such a neat way, than it is highly improbable that some form of spurious chars are in the way.
That said, I'll follow your advice, and will try the console.debug approach to see what happens.
I'll let you know. Thanks

Well, I've used Firefox and Safari to monitor network traffic and both don't have a "raw" view for the > JSON response.

Yes, I've noticed that, too. I think Chrome has an option to display the raw response. But you're probably right - if the browser can parse the JSON, WordPress shouldn't have any problems with it either.

Caught!
The problem, as suspected, was the "CamelCase" in my plugin name. For really mysterious reasons the wp_ajax_update_plugin function in the ajax-actions.ph file "sanitizes" the slug, which, among others, lowerize it (why do that in the plugins.php page and NOT in the update-core-php page? Who knows!!!!)
I've replaced the 'slug' => sanitize_key( $_POST['slug'] ), with a much more reasonable 'slug' => preg_replace( '/[^A-Za-z0-9_\-]/', '', $_POST['slug'] ), and everything is working fine.
So the culprit was Wordpress and your code is as innocent as a newborn baby ;-).
Thank you again for your valuable help.

Side note: Chances are that WP core developers didn't notice that issue because all plugins on wordpress.org have lowercase directory names. WordPress is not designed to support installing updates from third-party sources, so this particular scenario probably never got tested.

Yes. In fact I've submitted a bug report, explaining the whole issue, along with a fix proposal.

@caraffa I'd like to follow the bug report - could you post a link to it here ?

Never mind - found it: https://core.trac.wordpress.org/ticket/32465


Add support for WSL using clip.exe

In Windows subsystem for Linux, you can access native Win32 applications.  In particular, a nice one is clip.exe.  However, unlike xclip and xsel, it only accepts input from stdin, so the syntax is
echo "$yank_text" | clip.exe.  I tried editing the script and couldn't figure out how to get it to work when the piping is like this, so I can't just submit a PR.

btw I tried to add custom copy command to .tmux.conf: set -g @custom_copy_command '/mnt/c/Windows/System32/clip.exe' it doesn't appear in widows clipboard.

@fpqc PR #79 has been created for this but it seems like this repo is not actively maintained.

PR #79  is merged.  Closing.
And it is actively maintained again. :smiley:

ll

I have a method to solve it:   using C-z to make the pane to whole screen. Next , press 'Shift' button and using mouse copy which u want to copy. 😄

@BetterLuke: So was #79 not needed?

@BetterLuke what terminal emulator do you use? Is it the native conhost bundled with Windows or Mintty based terminals (like wsltty / wsl-terminal)?

what? this was good. dont listen to betterluke he's just wrong.
He doesn't understand the point of this script at all.

@lukewang1024 I'm using he native conhost bundled with Windows.

Right. I have also tested it in Mintty-based terminals and it works too. Basically holding SHIFT will trigger the terminal level text select / copy instead. Nice trick but it has nothing to do with what this issue does.

@BetterLuke  Thank you :) you made my night. Been looking for some workaround until this is fixed..


Improved documentation (to avoid error)

Following the instructions to install the monorepo builder, I executed:
vendor/bin/monorepo-builder release v7.0 --dry-run

And I got an error:
 [ERROR] There are no release workers registered. Be sure to add them to "monorepo-builder.php"

So I added the release workers in that same step

Thank you :+1:


feature: Home Page + Trailer Page

Has a couple of things:

The main page featuring every course
A trailer page containing the trailer video; data for the trailer was retrieved via an API call to the backend.
A component for videos that functions in both portrait and landscape modes.
Also has fixes from the pull request #14

just take care of lint errors


Docs about flashing with battery plugged in

Since there is a rework I'll just put the docs in this issue. You can copy the markdown and add me as a co-author to the commit plz:

When disabling WP by disconnecting the battery, do not flash firmware while the battery is disconnected. Your Chromebook could get bricked there is a power failure or you accidentally unplug your Chromebook while flashing firmware. Use these steps to minimize the chance of bricking:

Make sure your Chromebook is in developer mode
Shut down your Chromebook
Open up your Chromebook
Disconnect the battery from your Chromebook
Lightly cover the Chromebook with a lid but do not securely close the lid, because you will reconnect the battery later
Turn on your Chromebook with the battery disconnected
Open crosh (Ctrl + Alt + T)
Enter shell (type shell and press Enter in crosh)
Run

sudo flashrom --wp-disable
sudo flashrom --wp-range 0 0

If the command sudo flashrom --wp-range 0 0 fails, try doing
sudo flashrom --wp-range 0,0

instead
10. Check that WP is disabled with
sudo flashrom --wp-status

Shut down your chromebook
Reconnect the battery
You can now securely close and screw the lid
Turn on your chromebook
Flash full ROM using mrchromebox's firmware utility script

Do we want this? If so this should probably go under advanced

It's not advanced because it's the most common method of disabling write protect. It is more complicated than just running the script while it's unplugged, but it is safer. Putting this in advanced could cause less people who want to use the battery method to see it and could potentially lead more more bricked devices compared to just putting this in the WP section.

And a noobie not fully following the instructions before rebooting would do the same thing

@ethanaobrien Do you mean "people are too dumb to follow these instructions so put them in the advanced section?"

Yeah

Closed (stale)


Release Google.Cloud.Run.V2 version 2.6.0

Changes in this release:
New features

Support mounting NFS and GCS volumes in Cloud Run Jobs and Services (commit e85fcd7)
Support specifying a per-Service min-instance-count (commit e85fcd7)
Support disabling waiting for health checks during Service deployment. (commit e85fcd7)
Allow disabling the default URL (run.app) for Cloud Run Services (commit e85fcd7)

Documentation improvements

Clarify some defaults and required or optional values (commit e85fcd7)

Created release for Google.Cloud.Run.V2-2.6.0


create submission of ID3

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:missing

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:missing


where is the getting started document ?

do you have a real time chat room ?

Quick start doc can be found here http://pyscf.org/quickstart.html


I just changed

I changed on Delphi hello(i) to hello(i+.. errors
and to i+1 errors

While you type, you should see errors in the bottom panel, because halfway the converter encounters invalid code.
The example code is simple enough that the built in DWS compiler can even run the resulting pascal code once you're done typing.


This repo  is not archived

Open bugs for years, no releases and no one cares really about, I think we should edit the README, saying "please consider using https://github.com/redtide/sddm-conf " and  archive it.

Still valid?

I removed it now from weblate.

Objections to archiving it finally? Edit the README first?


Refs 3539: update yummy

Summary
Updates yummy to latest version. Includes fix where repo comps with certain file extensions or of group_gz data type were not detected.
Testing steps
These repos should be able to be introspected successfully:

http://yum.theforeman.org/pulpcore/3.16/el7/x86_64/
http://yum.theforeman.org/pulpcore/3.16/el8/x86_64/

Checklist

[ ] Tested with snapshotting feature disabled and pulp server URL not configured if appropriate

https://issues.redhat.com/browse/HMS-3539

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.

Looks good