What happens when two users each try to enroll a repo in Minder?

Is your feature request related to a problem? Please describe.
I'm Evan, and I'm responsible for https://github.com/foo/bar.  I've just heard about Mediator, and it sounds awesome, so I sign up and enroll foo/bar.  Unbeknownst to me, my co-maintainer Greg has already signed up foo/bar for Mediator.  What happens?  Can I get my own set of Mediator settings, and then set different values than Greg?  Do I get blocked?  Do I get added to the org?  Does Greg need to do that?  What if Greg did it a year ago and dropped off the face of the planet?
Alternate story:
I'm Evan, and I'm responsible for https://github.com/foo-org.  I've just heard about Mediator, and want to get all that goodness for my organization.  Unbeknownst to me, Greg in my organization has already signed up for Mediator (go Greg!), and has applied it to the repos he has owner control over in https://github.com/foo-org/awesome-dependency.  What happens in awesome-dependency when I apply policy that conflicts with Greg's?
Describe the solution you'd like
My personal preference: Given that I also have administrative control over the repo, I should be able to "adopt" myself or the repo into a common Mediator instance without necessarily needing to go find Greg to get permissions.
Describe alternatives you've considered
We could block Evan from controlling the repos in question.  This might be particularly annoying in scenario 2, as we'd need to get Greg to "renounce" awesome-dependency so Evan can take it over.
Additional context
@lukehinds suggested it would be good to file an issue to work through this.  @eryn-muetzel might also have some customer perspective.
Acceptance Criteria
We have a doc that outlines the answers to the questions at the top.

I think the GitHub App and other stuff generally fixed this.


Testing issue for comments

Testing this 'issues' option for commenting purposes. Is this where we want to leave editing comments to each other?

Testing replys

Hello! Thanks for the test :slightly_smiling_face:
Should branches and Pull Requests be used, then it would be most convenient for such conversations to occur there in the Pull Request, prior to work being merged in.
Regardless, yes, any "issues" with work that's been merged may certainly be discussed here!


apparatus rdg with src attribute

When a reading has a source attribute, the XSL doesn't render the bibliographic referene doesn't appear. When using a <note> inside the <rdg> the bibliographic reference renders correctly
see MedCY044.012
Need to decide if date should be displayed when there is more than one bib entry with the same author
Ex below has several ways testing what works now. Last line doesn't render

<app loc="1-5"><rdg>Ἰγνάτιος:</rdg> <note>Buckler</note> <rdg>ΙΓΝΑΤΗΟC: </rdg><note>Sacopoulo</note></app>
               <app loc="other_case"><rdg>Ἰγνάτιος:</rdg> <note>Buckler, Sacopoulo</note></app>
                   <app loc="alternative"><rdg source="#Buckler1933">Ἰγνάτιος</rdg></app>

assume we don't need the date

I have made a temp fix at lines 220–224 of htm-teidivapparatus.xml. Please delete these lines when you have implemented the above fix (presumably in the template named "sources" in the same file).

The Epidoc stylesheet handles <rdg> and <lem> that have a @resp attribute. I've modified the stylesheets so the display also works for when <rdg> has a @source attribute. Do we also want to handle <lem> when it has an @source attribute? As it now stands, the display doesn't include the year and it isn't a link.
This script is applied with the parameters $parm-edn-structure=('inslib', 'sample') or $parm-leiden-style = 'medcyprus'" I wonder if I should break out medcyprus separately given that we have changed the display for both @resp and @source quite a bit - no link, no year.
I note that we reserved judgement about displaying the year when there is more than one citation by the same author. There don't seem to be many.
Work mostly happening in: htm-teidivapparatus.xsl at l. 211 (template matching <rdg>) and at l. 262 (template matching @resp and @source)


Fix docker command in the readme

Fixes #67
Fix docker command in the readme file. Use volume instead of piping seems both work on linux & windows.
@NicolasMassart thx for the project.
Could you please check this?

This doesn't enable check from STDIN (standard shell input).
But I wonder if STDIN worked at all...
See https://github.com/tcort/markdown-link-check/blob/master/README.md#check-links-from-standard-input
It doesn't work either.
Maybe it would be nice, as part of this PR, to remove this part of the readme too?
#### Check links from standard input

    cat *.md | markdown-link-check

markdown-link-check CLI script clearly doesn't accept a markdown content as input, but only file paths and URLs:
.arguments('[filenameOrUrl]').
Then the test is only on if filenameOrUrl contains http or otherwise it's considered a local file. No markdown parsing at this stage.
The markdown through STDIN feature doesn't exit at all for what I can read.
Do I miss something?

This doesn't enable check from STDIN (standard shell input).

No, it does not. The whole idea is to replace STDIN reading with mounting a current directory in docker and read a file as usual via path /tmp/your_md_file.

The markdown through STDIN feature doesn't exit at all for what I can read.

Yes, that is also my understanding.
You're right, cat *.md | markdown-link-check gave me:
module.js:549
    throw err;
    ^

Error: Cannot find module 'chalk'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/mneverov/Projects/other/markdown-link-check/markdown-link-check:5:15)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)

removed this line from the readme, thx.


End time not used for automatic checkout when typical stay is 0 minutes (instead: 15 minutes)

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for Android only (? cannot check, no iOS)
[X] Bug is not already reported in another issue

Technical details

Device name: Google Pixel 3 XL
Android version: 11
App version: 2.0.4

Describe the bug
An event of type "Event" mandates that the owner sets a "Start" and an "End" date, otherwise it cannot be saved. However, the "Typical Stay" value is pre-set with "00:00 hrs" and can remain on this value.

When now a guest checks in to this Event, then the information about the "End" of the event is completely unused, instead an automatic checkout time of 15 minutes (sic!) is proposed:

Due to this, the clients are automatically checked out after this time.

Guests can of course later on adapt this time, but I fear this will happen only seldom.
Therefore I consider this as a bug, not as a Wishlist item. It will cause unwanted early check-outs and therefore missed notification targets.
Steps to reproduce the issue
Ups, already given above.
Expected behaviour
I propose that when the owner of the "Event" specified no "Typical Stay" value (which is currently the default), then the guest CWA shall use the "End" time of the event for automatic check out.
To be honest, this is IMHO the only reason why the "End" date is maintained at all (the "Start" time at least has the effect that guests, when checking-in earlier, get a message "The event has not yet started, do you want to check in anyway", or similar).
And maybe "Event"s shall not have a "Typical Stay" at all. For them, we already have events of type "Place".

Internal Tracking-ID: EXPOSUREAPP-6764

iOS:
"Place": Typical stay is pre set to 2h, it can't be 0.
"Event": Typical stay can be deactivated, if it's activated it's set to 15 min.
-> Android should be aligned to this, I guess.

@jkrwdf
Thank you for the report. We have notified the developers about this by creating an internal Jira ticket EXPOSUREAPP-6764.

Corona-Warn-App Open Source Team

@jkrwdf @Ein-Tim
iOS and Android should behave the same way now. Please check if the issue is fixed.

Corona-Warn-App Open Source Team

The CWA version available in Google Play Store for me as Android user is 2.4.3 and shows the same behavior (Even default "Typical Stay" is 00:00, and when not explicitly changed by even owner, CWA clients check in with 15 minutes) as reported on April 24th.
As the "Fix 2.3" label was assigned, I classify the check as "Not successful".

I also can confirm: the issue is not fixed using CWA 2.5.1
Reproduced:

Created event with regular check-out time set to 00:00
Scanned QR code
Automatic check-out time is pre-set to 00:15.

I think the problem is here that it shouldn't be possible to create an event that has a typical stay time of 0 minutes. It should also not be possible to say that you want to automatically get checked out after 0 minutes.
At least this is the behavior under iOS, it's not possible to have a typical stay time of / automatically check out after 0 minutes.

@jkrwdf @vaubaehn
Thanks for checking. I'll forward this to the dev team.

Corona-Warn-App Open Source Team

@Ein-Tim

I think the problem is here that it shouldn't be possible to create an event that has a typical stay time of 0 minutes. It should also not be possible to say that you want to automatically get checked out after 0 minutes.
At least this is the behavior under iOS, it's not possible to have a typical stay time of / automatically check out after 0 minutes.

and also referring to https://github.com/corona-warn-app/cwa-documentation/issues/657 :
My understanding how "events should work" is as follows:
If you set start time and end time, but you deactivate "typical time to stay" for an automatical check out before the end time as the event creator, then a user should be checked in when scanning the QR code, and be (automatically) checked out at the end time of the event.
This is useful where you have "short" events with a defined timeframe, where users usually don't go out before end time, e. g.  a movie in a cinema, a school lesson, etc.
But if you are creating a somehow "longer" event with a definded timeframe, e. g. a vernissage that starts at 10:00 and ends at 18:00, it can be useful to enter a "typical staying time" when you have experiences how long your guests usually stay, e.g., 2:00 hrs. A user who accepts the automatic check out after 2:00 hrs should be checked out automatically at 16:00 when he checked it at 14:00. A user who checked in at 17:00 and accepted the automatic check out after 2:00 hrs should nevertheless be checked out at 18:00 as the vernissage ends at this time. A user who checked in at 11:30 but did not accept the automatic check out after 2:00 hrs should also be checked out at 18:00 accordingly.
The question is now, how to "deactivate the automatic check out/typical staying time": in Android I would assume it's done by setting the "typical staying time" to 00:00. If this is not possible in iOS, I would think it's a CWA iOS bug.
The Android bug here is, that an automatic check out/typical staying time is set to 00:15 after scanning QR code, even it was set to 00:00 when creating the QR code.
So, probably it's an interesting mix-up of different bugs in iOS and Android CWA?

Hi @vaubaehn
Thanks for your detailed and long comment! I agree with you in all points & this is also how I would expect the event-registration feature to work, but apparently it is different. Everything what follows is only tested under iOS on app version 2.5.

If you set start time and end time, but you deactivate "typical time to stay" for an automatical check out before the end time as the event creator, then a user should be checked in when scanning the QR code, and be (automatically) checked out at the end time of the event.

Only if you create a QR-Code of the type "Event" you can deactivate the automatic check out during the creating process. However, if a user scans the QR-Code, he still has to enter after which time he wants to be automatically be checked-out. There is no way for the user to be checked-out at exactly the end time of the event, since there is no option "Check me out when the Event ends".

[…] user who checked in at 17:00 and accepted the automatic check out after 2:00 hrs should nevertheless be checked out at 18:00 as the vernissage ends at this time.

This is, again, how I'd expect it to work, but it doesn't. In the example the user would currently still get checked out after 2 hours and not at 18:00h.

A user who checked in at 11:30 but did not accept the automatic check out after 2:00 hrs should also be checked out at 18:00 accordingly.

As said above, there is no way for a user to "not accept the automatic check out" currently. The user would have to manually enter the hours until 18:00h.

The question is now, how to "deactivate the automatic check out/typical staying time": in Android I would assume it's done by setting the "typical staying time" to 00:00. If this is not possible in iOS, I would think it's a CWA iOS bug.

Mh, yeah. But it would be a bit difficult for the user to find this. I think it's correct that you can't set a typical staying time of 00:00. There should be an extra option "Check me out when the Event ends" (or similar), which the user can activate instead).
Under iOS, if you use the picker and set it to 00:00, it jumps to 00:15 min and takes this value.
Under Android I could imagine that it also takes 00:15 min, but does not show this to the user correctly.
The only situation where I think the start and end time is really used is this:
You scan a QR-code of an Event before the Event starts or when the Event is already over.
Then the app shows this:

Not yet started
Already over

Hi @Ein-Tim ,
thanks for your valuable testing!
For me it's not clear at the moment, if there are simply functions not implemented that could/should be expected to work (otherwise "event" in contrast to "place" does not make much sense imho), or whether there are a series of bugs?
@dsarkar and @heinezen
Could you help us out here? How should it work actually?

Dear @jkrwdf, dear community,
thanks for reporting and discussing this issue. With release 2.27 this issue has been resolved, therefore this issue has been closed.
Best wishes, DS

Corona-Warn-App Open Source Team

Nice to have you back @dsarkar!


Edit single Cell without rendering the whole row

Feature Description
A way to update a single cell
Use Case
I have a table with data that changes every sec.
most of the columns are not changing, but 1 column particular is changing frequently .
when I update the column, the whole row is rerendring.

This issue is lacking a reproduction. I suspect you are updating the data directly, and the table has no way of knowing that the cells should be re-rendered. Instead, the data source data should change and emit to the table


Inference code for lanegraph2seq seems wrong

Hi! I found that the test/inference code in RoadNetwork/rntr/ar_lanegraph2seq.py is the same with the test/inference code in RoadNetwork/rntr/ar_rntr.py, which is for RNTR instead of Lanegraph2Seq.
Would you please update the test/inference code for Lanegraph2Seq? Thank you so much!

I'm currently addressing the dataloader timing issue. If necessary, I can prioritize adding the inference code within the next few days.

@VictorLlu
Thank you for your update and reply.
Can you please also add the code for performance evaluation recently?
BTW, when running the training code, I print time for data preprocessing (time1), forward propagation (time2) and back propagation (time3) in mmengine/model/wrappers/distributed.py when training with 8 GPUs. It seems that the major cause for the long average batch time is the extremely slow back propagation in some iterations.

You can refer to #3 for details. I hope this can to some extent help you.

@ZJWang9928 @VictorLlu Hello, I am a little confused. There is no train.py and dist_train.sh files. How to run the code....

Thank you for your contribution. I've implemented the inference code for AR-RNTR first. You can utilize test.py, or it will automatically perform an evaluation after every 50 epochs.


fix(clustering) potential leaking for DP wRPC

Summary
It may leak when an error happens, and we recreate the wRPC connection. The previous one is not closed (instead, the WebSocket connection is closed. Probably a typo.)

Coincidentally it won't cause problems in most cases.
There are 2 threads, one for WebSocket listening and one waiting (on a queue) for the user to call RPC.
The 1 thread encounters errors if the WebSocket connection fails, and it sets the closing flag and exits. The other thread exits when seeing the flag.
This problem occurs when I wrote a unit test, with a lot of pairs of connections (and not close them).


Add/Remove Users to Orgs

Was needing an easier way to add existing users to an org. This simply just calls the Cloud Foundry's API, associate user with an organization by username and remove user with organization by username.
Some things to note is that this will fail if the user does not exist in Predix.

Made the requested changes. Added named roles let me know your thoughts on that.

Looks good, just want to test it out a bit myself and then can merge.


API: Add client provider interface for FileIO

This doesn't really buy much as an interface so probably isn't worth it.


davinci-resolve: .desktop file missing

Describe the bug
Follow up to https://github.com/NixOS/nixpkgs/pull/152113.
This might help: https://github.com/NixOS/nixpkgs/pull/152113#discussion_r795106901
Steps To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
Additional context
Notify maintainers
@jshcmpbll

try
postInstall = ''
install -Dm644 share/DaVinciResolve.desktop $out/share/applications/
install -Dm644 share/DaVinciControlPanelsSetup.desktop $out/share/applications/
install -Dm644 share/DaVinciResolveInstaller.desktop $out/share/applications/
install -Dm644 share/DaVinciResolveCaptureLogs.desktop $out/share/applications/
install -Dm644 share/blackmagicraw-player.desktop $out/share/applications/
'';

ref https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=davinci-resolve

Fixed in #240549


Fixes some performance issues

Fixes some performance issues:

response to AddBulkFiles was delayed until after all parsing had completed
reduce logging in TaskProvider by default
Also fixes a null reference crash when hovering over text.

Feel free to hit merge if no concerns.


Fix app crash when biometrics lockout on ios 13

When biometrics lockout on ios 13. It's not show native dialog, it's crash application.

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I signed it!


[$50] Move the file-level comment button to the toolbar on the top

Describe the bug
Currently, the file-level comment button is in the hover of the inline commit. We should move it to the toolbar on the top so that user can easily add/edit/delete or view the file-level comment.
Note: the 2nd placement submission has fixed this issue, you can refer the code.
To Reproduce
Steps to reproduce the behavior:

Go to the GitLens options
Click on the search button to open the new UI
Select one or several commits.
Click one file of the diff window and open the diff details.
Hover on a commit.
See the file-level comment button.

Expected behavior
We should move the file-level comment button to the toolbar on the top.
Screenshots
The original button is here

We should move it here

Desktop (please complete the following information):

OS: All
Browser N/A
Version All

Contest https://www.topcoder.com/challenges/30072347 has been created for this ticket.This is an automated message for billsedison via Topcoder X

Contest https://www.topcoder.com/challenges/30072347 has been updated - the new changes has been updated for this ticket.This is an automated message for billsedison via Topcoder X

Hi @billsedison .
I want to work on this issue, but I can't self assign it.
What I need to do to work on this issue ?

@narekcat Sorry. Let me check the GitHub setting.

@billsedison Ok, then I'll start to work on it.

@narekcat
I can't resolve the issue on GitHub, so I moved the ticket to GitLab, please read this announcement. https://github.com/billsedison/vscode-gitlens/issues/1
Sorry for the inconvenience.

@billsedison Ok, no problem.


Add Profile: TenzDelek

Profile Submission
Please paste a screenshot of your GitHub profile here.

Profile Category
Please select one or more categories for your profile:

[x] Badge 🎖
[x] Minimalistic ✨
[x] Dynamic 🔄
[ ] Icons 🎯
[ ] Backgrounds 😎
[ ] GIFS 🖼
[ ] Game Mode 🚀
[ ] Code 👨💻

Next Steps
Once you've filled out the information, please comment below:
@all-contributors please add @tenzdelek for review

@all-contributors please add @TenzDelek  for review


Firebase Database plugin (5.4.0, IOS, .Net 3.x) - InitializationException:  Firebase app creation failed.

When building for IOS with the Latest Google Play Games 0.9.52, Latest Firebase Database (5.4.0), And Latest Admob plugins for Unity, Application crashes when trying to create Firebase Database Reference
Here is a debug from Xcode:
ERROR: Unable to configure Firebase services: Google Analytics for Firebase version (50200000) does not match with Google App Measurement (50300000) version. Please update.
Unable to configure Firebase services: Google Analytics for Firebase version (50200000) does not match with Google App Measurement (50300000) version. Please update.
(Filename: /Users/builduser/buildslave/unity/build/Runtime/Export/Debug.bindings.h Line: 43)
InitializationException:  Firebase app creation failed.
at Firebase.FirebaseApp.CreateAndTrack (Firebase.CreateDelegate createDelegate, Firebase.FirebaseApp existingProxy) [0x00000] in :0
at Firebase.Database.FirebaseDatabase.get_DefaultInstance () [0x00000] in :0
Please, help.

I and my College have also encountered the issue with gem and cocoapods, namely, if the previous version of the cocoapods was installed and you try to install it once again (to update), in some weird cases  Unity will not link the Pods to the Xcode project.  The way to fight this issue is to remove the Xcode, then: gem unistall --all (with all of the dependencies), then re-install with:  sudo gem install -n /usr/local/bin cocoapods, and after do : pod setup
Then build the project for the Xcode, and try to build in Xcode.
This indeed helped to make Xcode build, but when launched on the device - there is always  InitializationException: Firebase app creation failed.

Just found out:
IOS, Mac Mojave, Unity Editor 2018.2.14f1, .Net 3.x
using the Empty project
install only Firebase Database 5.4.0
Make a simple script to test database

There is always a warning that there is no GoogleServices... .plist and googleservices.json, but they are inside the Assets!
When you run it inside the editor, now it will give you the error - nof found Assets/StreamingsAssets/ GoogleServices... .plist, Assets/StreamingsAssets/googleservices.json ! Right after you will get an error with  Firebase app creation failed!

Hi there,
Sorry for the delay in getting back to you.  It seems like there has been a problem with the Cocoapod versions that the Unity SDK is being pinned to.  We'll try to get a new version of the Unity SDK out with a fix as soon as possible.
In the meantime, what you can do is, in the Assets/Firebase/Editor folder, there will be a handful of files of the form *Dependencies.xml (likely AppDependencies.xml, AuthDependencies.xml, and DatabaseDependencies.xml).  In each of those, there would be a tag for iosPod, which will have version="5.9.0".  If you update that to "5.11.0", that should hopefully fix your issue.
Thanks for notifying us of this problem.

@a-maurice  I will try it later today, for now, we have to go back and downgrade the firebase to 5.3.0, it is stable for us

@a-maurice  yes it worked! Thank you!

Great to hear.  And we have released a new version with it properly fixed, 5.4.1.
https://firebase.google.com/support/release-notes/unity#5.4.1
The fix ended up being slightly different, pinning the AppMeasurement cocoapod to 5.2.  The 5.11.0 fix I suggested might work for you, but we did discover a problem with that approach when trying to use Cloud Messaging, so I'd recommend swapping to the official fix if possible.


NominatimNlpBackend  https://f-droid.org/en/packages/org.microg.nlp.backend.nominatim/

I installed this app from F-droid. Someone gave me an advise to install this to fix a problem with the GPS on my Fairphone 3 with  e/OS.
I was supposed to help getting a better GPS signal but it does not help. How do I uninstall this? I can't find it in the app list.
Thank you very much for your help and time
Affected app
Name: [e.g. WhatsApp]
Package id: [e.g. com.whatsapp]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System
Android Version: [e.g. 10]
Custom ROM: [e.g. LineageOS 17.1]
microG
microG Core version: [e.g. <IP_ADDRESS>915]
microG Self-Check results: [e.g. All ticked]

Additional context
Add any other context about the problem here.

It is in app list as "Nominatim Geocoder Backend".

Thank you very much. I am sorry can't find it in the app list.  I have it installed on my fairphone with e/os. Can I download adb to the phone?

For adb install Android SDK Platform Tools: https://developer.android.com/studio/releases/platform-tools

For adb install Android SDK Platform Tools: https://developer.android.com/studio/releases/platform-tools

Thank you very much @JonnyTech I did install adb. Connected the phone through PTP. But when I run the uninstall command as given by @ale5000-git
adb uninstall org.microg.nlp.backend.nominatim
This error message is what I get.

As per the message shown on the screen:
.\adb uninstall org.microg.nlp.backend.nominatim
Note the .\ at the start of the line.

Sorry overlooked it. Thank you very much.

As per the message shown on the screen:
.\adb uninstall org.microg.nlp.backend.nominatim
Note the .\ at the start of the line.

I am one step futher thank you.
Now it does not recognize Nominatim which is definitely installed on the phone.
The Phone itself gets recognized ones connected.

It is just saying that it don't see the phone.
adb don't use PTP, you should enable adb in developers options on the phone.
You can disable it again at the end, after running the adb command.

It is just saying that it don't see the phone. adb don't use PTP, you should enable adb in developers options on the phone. You can disable it again at the end, after running the adb command.

Yet one step further. Enable debug.
Haha this seems such a simple command... but sometimes simple thing become complex...
Do you have a new suggestion perhaps? Thank you and have a good weekend!

.\adb devices
Your phone should be listed as connected.
.\adb shell cmd package list packages
All installed apps will be listed.
.\adb shell pm uninstall -k --user 0 app-from-list
Uninstall app from previous list.
.\adb shell pm uninstall -k --user 0 org.microg.nlp.backend.nominatim
Make sure org.microg.nlp.backend.nominatim was on the list, modify package name if required.

.\adb devices
Your phone should be listed as connected.
.\adb shell cmd package list packages
All installed apps will be listed.
.\adb shell pm uninstall -k --user 0 app-from-list
Uninstall app from previous list.
.\adb shell pm uninstall -k --user 0 org.microg.nlp.backend.nominatim
Make sure org.microg.nlp.backend.nominatim was on the list, modify package name if required.

Thank you very much. I will check it right away and let you know asap.

this command was not recognized. .\adb shell pm uninstall -k --user 0 app-from-list

Sorry, I thought it was obvious that app-from-list should have been replaced with the app name from the list.

Happy camper here.

Glad to have helped. Can you please close this issue now?


Header/#31

Description
Header first version
Media queries come on next version
Please provide a short summary explaining what this PR is about.
Design the header for both guest and registered users.
Fixes # (issue)
#31
Screen shot:
Guests see this:

Registered users see this:

How to test?
Can be seen on the homepage now. (localhost)
Please provide a short summary how your changes can be tested?
try to run app and check out this: http://localhost:3000/
The storybook can be checked as well.
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] This PR is ready to be merged and not breaking any other functionality

@arya56 æets get this merged in soon. I think @valmdz  review is good, but let's address mobile later on. I think some of the css tricks she suggested can be applied, but if not, please write back to us.
Cheers,

@pdsorensen  applied some of the suggestion changes.


Pasting Multiple Tags

Hi!
I'm looking to adopt this for a project and I'm wondering what support is like for pasting multiple tags. I didn't see anything mentioned in the documentation, and it looks like pasting in a comma-separated-list is still treated as one tag. Eg: I paste "blue, yellow" and it's treated as one tag instead of two. Is there a thing that is configurable or would be an easy change?

No configurable thing but this is the way I did it. I hope it helped.
<multiselect ... @tag="addTag"></multiselect>

export default {
    // ...
    methods: {
        addTag: function (value) {
            value.split(',').map((item) => {
                this.selected_tags.push({
                    name: item.trim(),
                });
            });
        },
        // ...
    }
}

Oh that'll work perfectly, thanks for the help!


View update skips winning revisions

Sadly it's hard to reproduce in an isolated environment.
If there are two conflicting revisions, and one of those is deleted, the view update skips the new winnig revision so the document isn't indexed.
For example:
seq  | doc_id | revid | parent | current | deleted
-----+--------+-------+--------+---------+--------
2750 | 172    | 1-77  | NULL   | 0       | 0
2751 | 172    | 2-1a  | 2750   | 0       | 0
2752 | 172    | 3-0c  | 2751   | 1       | 1
2753 | 172    | 1-6b  | NULL   | 0       | 0
2754 | 172    | 2-1a  | 2753   | 0       | 0
2761 | 172    | 3-d7  | 2754   | 0       | 0
2763 | 172    | 3-06  | 2754   | 1       | 0
2764 | 172    | 4-80  | 2761   | 1       | 1

(I cut the revid)
"3-d7" and "3-06" are conflicting, "4-80" deletes the branch "3-d7" is in, so i expect "3-06" being the winning revision and thus passed to the view function.
Since all rows of the same doc_id are skipped at:
https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/store/SQLiteViewStore.java#L351
"3-06" is also skipped, so the document 172 isn't indexed at all.
For me this is fixed by not skipping the rows if the current revision is deleted.
while ((keepGoing = cursor.moveToNext()) && !deleted &&
                        (cursor.isNull(0) || cursor.getLong(0) == docID)) {

I do not understand why the following block (#L358-L391) doesn't find "3-06" as winning revision.
Since it's only 10ish chars, i hope it's ok to not send a PR. :)

Hi @saubaer, Thank you very much for your issue report.

Hi @snej,
Can you help me to understand correct behavior? Is current behavior correct, or as user suggested, should other branch's leaf revision be indexed?
Thanks!
Hideki

There's some tricky logic in the index updater for handling a revision that updates a conflict. In a case like the one shown above, the new revision doesn't get indexed (it's a deletion) but it has the side effect of causing a different revision to become the winning one. In the Objective-C code this is handled in CBL_SQLiteViewStorage.m:445-453.
I did a git blame to find out where this code comes from — it was a fix for https://github.com/couchbase/couchbase-lite-ios/issues/578, which you filed as a copy of https://github.com/couchbase/couchbase-lite-android/issues/494. So now I'm confused!

Maybe I can add to the confusion:
The case should be handled by the block:
https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/store/SQLiteViewStore.java#L358-L391
But that only checks for sequences lower than "minLastSequence" so for sequences or revisions not part of the first query ("cursor").
So if the view update would have been triggered for seq. >= 2764 that later block would pick up the seq. 2763 as winner.
In my case, the view update is triggered with a "minLastSequence" of  <= 2763, so the new winning revision "3-06" is in the result list of the first query ("cursor") and is not seen by the 2nd query ("cursor2").
Hope this makes sense.

Hi @saubaer,
I wrote Unit Test to reproduce the problem. https://github.com/couchbase/couchbase-lite-android/pull/706 . However I could not reproduce it. I believe the fix for the ticket covers this scenario. If you have time, could you please review my test case?
Thanks!

Issue is not reproducible with unit test that tries to reproduce this. I believe this scenario is covered by fix of  the ticket. I will close this ticket.


Remove artifacts of previous commit

I think I've found two artifacts introduced by commit 4035f43a2f0c529eb1b32735af800f19c7d7a9e7.
Also, I would suggest merging comparisons with 'in', as this is probably faster and less verbose.
Is there a reason that self._key gets defined outside of init()?
(cf. https://github.com/dftbplus/hsd-python/blob/master/src/hsd/parser.py#L222)

Codecov Report

Merging #9 into master will not change coverage by %.
The diff coverage is 80.00%.


Bahasa Indonesia translation in p5.js website

Hi, I’m Tari
I’m interested in the Internationalization improvements project for the GSoC2022, as a native Bahasa speaker, I would like to work on a Bahasa Indonesia translation of the p5.js website. I would like to know if this could be an acceptable

Hi @untari, you can post your project idea on the forum to get some feedback there. I can't comment about future plans for translations as I'm not involved in the process but I am excited about including Bahasa as a new language to p5.js. I'm going to close this issue for now in favour of gathering feedback in the forum. Thanks.


Form Files

Is your feature request related to a problem? Please describe.
Currently, when I define a type there is no way to specify a form file field. In other words, I'd like to define an endpoint that can accept a multipart form file. The form file part of the request is a complex type containing a header with file metadata and the actual file content split by a delimiter which is also defined in the header.
Describe the solution you'd like
In a request type, I'd like to specify that there is a form field, say upload_file for instance that represents a multipart form file.
for example
@server(
  group: files
  prefix: /files
  middleware: CurrentUser
)

service files {
  @handler UploadFile
  post /upload (UploadFileRequest) returns ()
}

type UploadFileRequest {
  File []byte `form:"upload_file"` // This is the part that is not supported from what I can tell.
}

Describe alternatives you've considered
Defining a custom http handler. This works, but we lose the code generation and shared middleware of our other endpoints.
Additional context
Add any other context or screenshots about the feature request here.

If you want to parse the file byte stream in the form, you see zero-example repo

you can see   ,it may be sure you


docs(rh-timestamp): update changeset

What I did

Updated changeset to match previously released feature (code block)

@bennypowers maybe the indentation I had in my changeset cause it to become malformed? In any case, I just copied the formatting from rh-code-block. I'm hoping that'll do the trick.


⚠ Bofrost.at has degraded performance

In c5527c9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1478 ms

Resolved: Bofrost.at performance has improved in 9aacc9a after 6 minutes.


Feedback for "Security Events Logged by the CF Services"

https://help.sap.com/docs/BTP/65de2977205c403bbc107264b8eccf4b/84acdc91c60c4f32b7c06cdbc442fdef.html?locale=en-US
The SAP Build Workzone Standard edition (SAP Launchpad) is not included in the table and I believe that it also logs audit related events.... See Launchpad documentation here: https://help.sap.com/docs/Launchpad_Service/8c8e1958338140699bd4811b37b82ece/d7b1a89c9a0f46bd9b1e2b6ca70add66.html?locale=en-US&q=Auditing#data-protection-and-privacy

Dear ajmaradiaga,
Thank you for your feedback!
The reason Launchpad and some other services, which also log audit related events, are not included in the table is because they have not created the required Auditing and Logging Information page for their service. There are jira issues created for each service.
I reminded the colleagues from Launchpad to complete this task. I'll update the table as soon as the page is available.
Best wishes,
Petia

I close this issue as Petia already responded that the problem is addessed and the documentation is to be updated within next 2 weeks.


Flag machine specific configurations like paths as machine scoped

I am developer from VS Code and we introduced Machine Scope for configurations in 1.34 which will help in remote set ups.
I would like to encourage extension authors to adopt to machine scope if you have any settings that you think they are scoped to machine like xmlTools.xqueryExecutionEngine
For example, If you have settings that allow users to customize an executable path that exists on the machine in which extension is running, then you can tag such settings as machine scoped. This will let VS Code to read this setting from the location where the extension is running.

When extension is running locally, VS Code reads this setting only from User settings on the same machine.
When extension is running remotely, VS Code reads this setting only from Remote settings on the remote machine.

If the extension is running remotely in a remote set up, then this will help the extension not to read paths configured in local machine user settings and bail as the paths not exist in the remote machine.
Note that machine scoped settings are overridable only in user or remote settings.

related: #283


https://aka.ms/creatorkitdownload points to older version

Hello.
I was downloading the version linked of the creator toolkit from the page. The link is https://aka.ms/creatorkitdownload. The version/file downloaded is CreatorKitCore_1.0.20220604.1_managed.zip which is not the latest version of the toolkit on GitHub. The latest version is  CreatorKitCore_1.0.20220729.1_managed.zip
The aka.ms link should point to the latest version I would assume?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6b74154c-ce00-1e30-0806-9d9a1d5923ea
Version Independent ID: 6b74154c-ce00-1e30-0806-9d9a1d5923ea
Content: Creator Kit - overview (preview) - Power Platform
Content Source: power-platform/guidance/creator-kit/overview.md
Service: power-platform
Sub-service: guidance
GitHub Login: @denisem-msft
Microsoft Alias: demora

Thanks for your comment, @remyblok. This issue has been assigned to the doc author for review.


Enhancementrewrite framework used in Memo

Proposed changes
Issue Number: close #9627 , #9628
Details will be add later
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (No Need)
Has document been added or modified: (No Need)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Please enrich your commit log

Please enrich your commit log

done


matlab.tgz Should Reflect MATLAB Version Number

If there's already a matlab.tgz in /Applications there's potential for conflict. Not much work to have curl, tar and rm preserve the version number of MATLAB rather than using the generic name. Need to confirm what actually happens before addressing.

On second thought, the current way may be preferable because users would likely not have renamed the package matlab.tgz.

For standalone installer, let's keep things as is; for multi-installer version number are reflected.


HT space to image space

Hi @yanconglin ,
I would like to kindly ask how we can move from your HT space to an image space, also which of the corners of the image was set to be the origin(or it is the centre of the image?)
It will be kind to explain or provide code for this. Actually, I wanted to draw the lines obtained after the HT  module but it gets too complicated (from Rho, Theta I wanted to write a code to draw it)
Thanks in advance

To save memory, I used the centre of the image. See there for details: https://github.com/yanconglin/Deep-Hough-Transform-Line-Priors/blob/812d8c21e98e7b11b2e81b88faad8c8709cbd364/ht-lcnn/lcnn/models/HT.py#L48
say (x_theta, y_rho) is the ht bin you want, simple do vote_matrix[:,:, y_rho,x_theta] and then you have the line in images. This is the  binary mask of the line at  (x_theta, y_rho)..

Hi @yanconglin ,
Thanks a lot for your reply, May I ask you again what out[::-1] and y refers to in line 185 in the hourglass_ht.py and what are they expected to contain, Thanks again

"y" is the backbone features that are used in the line vectorization part.
"out" is a list where each element contains a tensor composed of  [lmpa, jmap, joff]. "[::-1]" means the order is reversed. Originally,  the out[0] is from stack 1, and out[1] is from stack 2. Now it is the other way around. In the follwing procedure, the predictions from the second stack are used, which means out[::-1][0]. IMHO, the HG model is quite complex, not so easy for beginners.

feel free to reopen this isssue if you have more questions.


pydantic > 2.1.0 is too restrictive

Restrictive Dependency
pydantic's minimum version was bumped from 2.1.0, which was released less than 3 weeks ago.
https://github.com/supabase-community/gotrue-py/blob/c2ed950516e4b9bee6eb8f066969049b9a361a28/pyproject.toml#L20
pydantic is a common dependency, particularly among API building libraries and this makes gotrue-py/supabase-py incompatible with those libs
For example, we are currently incompatible with FastAPI and LangChain

same problem as https://github.com/supabase-community/postgrest-py/issues/282

+1 on this issue. I'm seeing that a big draw for the Supabase Python client is its use in AI projects. That pydantic version restriction is causing some hiccups. It's making things incompatible with:
langchain
chroma
(and I'm guessing there are a few others)
For me, the essence of the Python client is to mesh Supabase into our Python workflows, especially in AI territory. I know a broad world of Python use cases, but this feels like a core one.


Improved shader restorer

https://github.com/mafaca/UtinyRipper/issues/434
Fixed Shader Restorer when reading from stream with offset would assign chunks the wrong offset
Fixed gap Shader Restorer constant buffer unused variables
Changed Shader Restorer constant buffer unused variables to use float4 arrays instead of int1 array

The were a few issues with the shaders without UAVs. The shader restorer expected streams passed to it to have a position of 0, and the latest change broke that assumption.
The SubProgram data is missing constant buffer entries for some variables. I had previously thought that the unused dummy variables were variables that the shader didn't use, but there are apparently shaders that read from those locations, so 'unused' is a bit of a misnomer.
There were some mistakes in calculating the unused dummy variables, so the shader would read from addresses with no variables assigned, the gaps between variables.
The shader models registers as float4 vectors, so changing the dummy variable type is a better fit for what the shader expects.

I enabled HLSLcc exporter. For #434 it works fine. Can't test it on other projects tho. Lets see if people find some other issues.


Create Postgres Index for AwardNumber

Create an index for the awardNumber field. This field is searched upon heavily using a large OR concatenation to take into the variety of NIH award Number formats.
Example of where the index will be most needed: https://github.com/eclipse-pass/main/issues/659

Done. There is now a trigram index in postgres which should help.


Add IServerAddressesFeature support

Fixes: https://github.com/aspnet/AspNetCore/issues/3843
This feature requires both new shim and new handler to work but is gracefully disabled otherwise.

Multiple IIS test failures?

Yep, I missed something. But in general PR ready for review.

Failures are AuthSamples tests


PascalCase or camelCase for public functions on modules

Does this document answer the question about the naming convention for public functions on modules? Should those be PascalCase or camelCase?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8c09996f-24ac-cf39-ef43-d7909cbc2b89
Version Independent ID: 1efdd523-8552-6421-e149-74f76826ef8f
Content: F# code formatting guidelines
Content Source: docs/fsharp/style-guide/formatting.md
Service: unspecified
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: phcart

I don't think it answers it directly, no. Module-bound public functions should be camelCase. This section technically covers it, but I don't think the example makes it clear. I can make a change.

Doesn't the F# component design guidelines article cover that?

let-bound values are often public when following traditional functional design patterns. However, generally use PascalCase when the identifier can be used from other .NET languages.

@svick It does, but I don't think people would expect to go to that article to find this guidance. The first push for these sets of articles were to get the CDG and other writings synthesized and published. Various cleanup efforts need to happen now.


Nginx with Modsecurity: POST request gives 500 error

Nginx with Modsecurity: POST request gives 500 error，use no 80 port ，no problem，Is there a way to solve this problem you

Hi @houhuijian,
We have a couple of issues opened regarding this very same subject. You can list those here:
https://github.com/SpiderLabs/ModSecurity/labels/Platform - Nginx
Inside the issues you will find discussion and possible solution to your problem.
Our proposal to solve all that issues is by the utilization of ModSecurity version 3 which is currently under development.


Feature Request - Session Cancellation Function

It'd be great to have a between node cancellation function for the tensorflow python session. In particular, if you call sess.cancel_graph() on a separate thread, then the tensorflow session on the main thread would be cancelled (raising an exception like the timeout exception). It would also be great for these cancellation signals to be caught between every tf operation, rather than only after particular operations like dequeue, etc.

Does tf.Session.close() not already satisfy this requirement? It cancels all concurrently running steps in a particular session, and these calls raise tf.errors.CancelledError.

Ah I wasn't aware of this feature. Thanks for pointing it out! I'll try it out.

Is there a way to keep the session alive but still cancel all the current steps asynchronously? I'm doing this in a distributed context, so keeping the session alive is necessary to continue training.

Is there any update on this?

Note that recreating session is normal part of distributed training. For instance, if you have a network delay and session.run times out, the session becomes invalid and needs to be recreated.

Automatically closing due to lack of recent activity. Since this issue is old at this point, please reopen the issue if it still occurs when tried with the latest version of Tensorflow. Thank you.


feat/monorepo

:recycle: Change package name (core -> react-crisp-toast)
:recycle: Refactor monorepo structure: Move packages to root directory as separate entities
:sparkles: Setup CI (github actions)
:recycle: [refactor] .
:recycle: [refactor] Change package name

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Cannot open one ADLS Gen1 account with an error

Storage Explorer Version: 1.8.1_20190524.1
Platform/OS: Linux Ubuntu/macOS High Sierra/Windows 10
Architecture: ia32/x64
Regression From: Previous release 1.8.1
Steps to reproduce:

Create an ADLS Gen1 account -> Try to open it in SE.

Expect Experience:
The created ADLS Gen1 account can be opened successfully.
Actual Experience:
Cannot open the ADLS Gen1 account with an error.

This issue has been fixed on latest build_20190524.5 of feature branch. So Close it.


Vertical overflow

Description
Setting css of a .container (in the site footer) and .dashboard .dashboard-navbar to 100vw causes a vertical overflow in Firefox (v112).
On Chrome(ium) it is mitigated with overflow: overlay, but it isn't supported on Firefox yet.
On the demo site it is visible when creating a repo https://try.gitea.io/repo/create
Propsed change: removing the set width of 100vw on these classes eliminates the need of hacking the vertical overflow on the body (with overflow-x: hidden;).
Screenshots

Gitea Version
1.19.1
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
MacOS 12.6.5
Browser Version
112

I do not see any scrollbars in https://try.gitea.io/repo/create with Firefox 112 on MacOS. What's your version/os?

Operating System
MacOS 12.6.5
Browser Version
112

Also in Firefox Developer Edition, 113.0b4, without extensions.

Just had a thought: as it doesn't happen on my Debian/KDE Firefox installation, where the scrollbars are kind-of-nonexistent, I'll check that out:

And sure enough, when scrollbars are automatic, the layout is all right. When set to "Always", the overflow bug is now visible.
Should I create a PR with a layout fix?

Ah, yes I see it as well with "always" scrollbars. Please open a PR so we can test/discuss further.

@silverwind Pushed. Thanks for taking a look at it!

Closed by  #24184


Feature/automated soft credits  ordered

Addressed items from the previous PR review.
Resolved test failures.
Updated Help Text per Work Item #W-025199.

Critical Changes
Changes
Automated Soft Credits
NPSP now supports several new automated soft credits. Previously, NPSP could automatically create soft credits for household members of a donor, for Honoree or Notification Contacts, and for donors in a Matching Gift scenario.
In addition, NPSP now creates automated soft credits for the:

Primary Contact on an Organizational Opportunity
Related Contact on a Relationship record with an Individual donor
Contact on an Affiliation record with an Organizational donor

Many organizations get by just fine with only Household soft credits. But some organizations have a donor base that requires a little more attention. Depending on the kind of organization you work for, you might have to accommodate donor advised funds, memorial gifts, planned giving, major donor portfolios, giving circles, crowdfunding, peer-to-peer fundraising, and much more. In many of these cases, you need to assign soft credits to one or more people, such as a board member, members of a family foundation, or simply relatives who don’t live in the same household.
Getting all of this right is important! Donors with more complex needs will expect that you are representing their family, businesses, associates, and financial institutions correctly, and that your organization is crediting the right people for the donations they've helped to bring in.
NPSP now lets you configure soft credit contact roles in multiple ways to help accommodate these different scenarios. And since NPSP creates the soft credits automatically, you can scale your fundraising efforts while continuing to honor the wishes of your donors.
Resources:

Automated Soft Credits
NPSP Logic for Creating Contact Roles
Configure Soft Credits

Issues Closed
New Metadata
Deleted Metadata

@joy-sfdo  Just got the following error saving an Opportunity.

One Relationship between the Primary Contact and another Contact (Carl Harvey in this example)
Edit the Opp and remove the Primary Contact and then assign "Carl Harvey" to the Honoree and Tribute Contact fields.
Click save.

TDTM_Opportunity: execution of AfterUpdate caused by: System.DmlException: Update failed. First exception on row 0 with id 00K19000001chCSEAY; first error: INSUFFICIENT_ACCESS_ON_CROSS_REFERENCE_ENTITY, insufficient access rights on cross-reference id: [] Class.UTIL_DMLService.updateRecords: line 202, column 1 Class.TDTM_TriggerHandler.processDMLAllOrNothing: line 241, column 1 Class.TDTM_TriggerHandler.processDML: line 221, column 1 Class.OPP_OpportunityContactRoles_TDTM.run: line 81, column 1 Class.TDTM_TriggerHandler.runClass: line 183, column 1 Class.TDTM_TriggerHandler.run: line 105, column 1 Trigger.TDTM_Opportunity: line 34, column 1

Then edited that same Opportunity add "Debra Knight" back as the Primary Contact. That saved and created a Primary OCR Role for Debra and a Solicitor OCR for Carl, but it deleted the Honoree and Tribute OCR roles that were created previously.

Thanks for the code review, Mike. I'll address the items listed and look into the error.

@joy-sfdo Found another issue this morning. Created a new Org Donation, specifying a Primary Contact. The "Contact Role for Organizational Opps" field in NPSP Settings is blank. The result was an OCR record marked as IsPrimary, but with a blank role:

@force2b -- That adds up. If the Role was not specified then the value would be blank. What is the expected behavior?
@mpusto -- What are your thoughts on the above issue - "blank OCR Role value when a value is not specified in NPSP Settings" @force2b found?

@joy-sfdo Found another issue where duplicate OCR's are created. In this case, I have defined "Donor" as the "Contact Role for Organizational Opps" in NPSP Setting as well as set up an Affiliation for Debra Knight on the Edgewire Account with the "Related Opportunity Contact Role" field set to "Soft Credit".
Opp showing dupe OCR's

The Affiliation:

I looked at how this is handled currently for Individual Opps and there is validation in NPSP Settings to prevent this scenario from happening. It seems like we should have the same validation for consistency. @force2b @joy-sfdo Do you agree?

@mpusto -- Sounds good. Please show me where the validation takes place and I'll update the code. Thanks.
Good catch, @force2b!

@mpusto and @joy-sfdo  NPSP Settings will already require a value for this new field, however the field will still be blank when this new feature is deployed into all existing customer orgs. I think we should auto-populate the new field with a value of "Donor" when the updated package is installed (via the Install Script).

@force2b -- If we auto-populate the field, I'd recommend the value of "Soft Credit" to distinguish it from Individual gifts that default to "Donor". Thoughts?

Looks like this is where the validation occurs: https://github.com/SalesforceFoundation/Cumulus/blob/master/src/classes/STG_PanelContactRoles_CTRL.cls
For a default I think @joy-sfdo's suggestion of Soft Credit is good.

Sounds great. Thank you.
@joy-sfdo There are two places that will need changing to support this:

UTIL_CustomSettingsFacade.configHouseholdSettings() - set the field to "Soft Credit"
STG_InstallScript.createNewDefaultCustomSettingRecords() - check to see if the field is blank and if so, update the value to "Soft Credit"

@mpusto -- Thanks for the link to the code. I was asking about the action/steps a User would take to generate the validation error.

Just leave the value blank in NPSP Settings.

@force2b  -- I just recreated the error. Thanks.

My bad! Essentially a user would go to NPSP Settings-->Donations-->Contact Roles, click Edit, then attempt to clear the value from the Organization Contact Role field (change to --None--) and click Save.

@force2b and @mpusto -- I see the error for Organizational Contact Roles being blank in my scratch Org.
@mpusto -- thanks for the steps.

@force2b -- I see that's the screen capture you shared earlier.

@force2b --
In UTIL_CustomSettingsFacade.cls the Individual value is set in configContactsSettings and the Organizational value is set there too. When spinning up a new scratch org sometimes the value of "Soft Credit" is not set in the Opportunity Contact Roles. Therefore, when a person navigates to NPSP Settings the Soft Credit value displays, but when they click edit the field becomes blank. I know you're resolving this issue.
I'll checkout --
STG_InstallScript.createNewDefaultCustomSettingRecords() - check to see if the field is blank and if so, update the value to "Soft Credit"

@joy-sfdo On the Affiliation object, the Related Opportunity Contact Role field has 'Soft Credit' 'Solicitor' as the out of the box options, but also 'Matching Gift'
Why 'Matching Gift'? This seems out of place and would need to be created as a defined OCR value if used, no?

@judisohn -- Thanks for following up. The value 'Matching Gift' is one of the field values specified in the ACs for the following Work Item --
https://foundation.lightning.force.com/lightning/r/agf__ADM_Work__c/a2x800000001k20AAA/view.
@mpusto -- Please see Judi's questions regarding the 'Matching Gift' field value.


add plugin:percol

Use percol to search&select history,

Great addition - I've added a couple of comments...

I added a feature to work with fasd.

The function does not work for me on OS X. Which OS are you on, and which version of sed are you using? The BSD version of sed bundled with OS X does not support the sed expression you're using:
sed -e 's/^\s*[0-9]\+\s\+//'

I think \+ is a GNU extension, and the \s does not seem to be recognized either.
I was able to make it work using sed -e 's/^[[:space:]]*[0-9]*[[:space:]]*//', which uses the POSIX character classes. Does that work with GNU sed as well?

sed -e 's/^\s*[0-9]\+\s\+//' works well with GNU sed!

Yes, that's what I'm saying, OS X comes with the BSD version of sed...

I still have an issue on OS X with this. Here's what I do:

Type "Ctrl+R" to start the percol history command.
I can now see the whole history and can filter, move up and down, etc.
Once I'm on an entry I like, I hit the ENTER key
The percol filter is closed, but my command line is still empty.

Somehow the selected entry is not put into the current line...
What am I doing wrong?

@nwinkler I tried your steps but everything works well on arch linux..

READLINE_LINE and READLINE_POINT seems not work well on mac.

True. READLINE_LINE and READLINE_POINT were added in Bash 4, while OS X by default has Bash 3.
Can you add a comment at the top of the file that will only work with Bash 4. I'll merge the PR after that.

You are right,there is no way to do same thing in bash 3.x.But maybe a bash version checking is better than comment.

...Opps, sort -V not work in OS X either.

We still have last problem...
echo 23 trter|sed -e 's/^\s*[0-9]\+\s\+//' get trter|sed on Linux but get 23 trter|sed on OS X.

Now everything works well in both Linux and OS X.And I squashed all commits into one.Ready for merging.

Thanks!


Support for .ts-files

Just the plugin I was looking for! I wish it would support .ts-files though, they do not seem to show up in the test explorer.

Please see the "Having Problems? Read this!" section:

The Node.js test runner only supports running JavaScript files. If you have a compilation step, you'll need to make sure that runs with sourcemaps so we can figure out where tests appear in your source code. For example, for TypeScript, set "sourceMap": true in your tsconfig.json.

Actually, the test runner supports .ts-files if you pass in a loader and the files explicitly, it just doesn't discover them:
node --test --loader tsx src/foo.test.ts

The above works well. Could there be a setting in this plugin to configure node-args so we can pass in loaders, and also to make the include-option a glob instead of just directories?

It seems like the loader is still very experimental (stability 1), and comes with a caveat:

When loading ECMAScript module loader loads the program entry point, the node command will only accept as input only files with .js, .mjs, or .cjs extensions; and with .wasm extensions when --experimental-wasm-modules is enabled.

Though what you said seems to contract that. Will need to play around with it, I haven't personally tried to use --loader/--experimental-loader before.

@connor4312 node test runner is experimental too, and still have a lot of bugs )
Seriously, ESM loaders are stable enough since 18.x (and much more usable after https://github.com/nodejs/node/pull/43772 in 19.6). A bit strange naming convention is not a problem due to '--test test/**/*.spec.ts'.
Your extension looks very promising, so please - don't introduce additional restrictions to the workflow (as we can see in many other, much more mature and popular extensions). Even if those restrictions are implicit, like inability to pass glob-based file list or configure node argv / NODE_OPTIONS (yes, I have to create separate script which mimics node binary to use eslint extension with latest node & yarn features), etc.
Thx!

@connor4312 may be stupid question - we can specify custom loaders & other node options (which is really cool), but we can't use .ts files due to extension doesn't look for them, right?

ah, right!

You can now do this, example config here: https://github.com/connor4312/nodejs-testing/blob/main/testCases/tsx/workspace/.vscode/settings.json


Almost there!!  Error:404: Page Not Found

Issue Description
I am using the generic Web credential template and can send emails to users. Users get the email and when they click on the link they get Error: 404: Page Not Found.
the link looks right - https://name.com/login?id=Qwhnfnlskd33
Reproduction Steps

Start King Phisher
Do something to trigger the crash

Environment Details

Host OS: Ubuntu 16.04 lts.
King Phisher Version: 1.5.1
Error Details / Stack Trace
Not sure what to put here..
If you are opening an issue regarding the client, copy text from the "Error
Details" of the error dialog in the client here. If you are opening an issue
regarding the server copy the relevant lines from the logs here.

Please review the FAQ 404 Error. You will want to use the jinja text the text of the link  to dynamically create the link for each user. This will allow them to view the landing page, and for you to collect the data for each unique email you send out.

I'm closing this issue because there hasn't been any activity in over a week.

Sorry - I have been swamped here at work. Everything is working fine