Multiselect size

Gretings,
In our application the user can set the size of the bootstrap 4 inputs from the project: small, medium, large.
In order to do that on this component we need to apply the following classes on the button (that has btn-select class on it): btn btn-sm, btn btn-md, btn btn-lg. Also we need to change the font-size, padding and min-width from it. (these are set default by the following css: .select .inlineBlock[data-v-1604b90e]).
Can you make a prop, or something that can give us access to customize/overwrite the button class?
Thanks!

Hi, does 4.2.0 resolve your pb ? you might need !important on some params

Thanks a lot, that was fast.
It works but i had to redefine the bootstrap classes with new names in my style.css and put !important on all. Might not be the best solution but it works and it's ok for now.
Thanks again!


Refactored code for handling tasks on master.

Did not really solve the problem, so current solution stays


feat(tenants): return value from withTenatnAgent

Has been boterhing me a lot, as it makes the function a lot more complex to use and thus I used getTenantAgent, but this way it's easer to use withTenantAgent, which always makes sure the session is closed

Great! About to raise a issue on this and saw this fixed. Thanks @TimoGlastra


[BUG] Duplicate GlobalKeys detected in widget tree.

Describe the bug
I have a page with QuillHtmlEditor that can be navigated from multiple places, that means I can have this page opened multiple times in the stack.
The problem is when that happenes the following message will appear:

The following GlobalKeys were specified multiple times in the widget tree. This will lead to parts of the widget tree being truncated unexpectedly, because the second time a key is seen, the previous instance is moved to the new location. The keys were:

[GlobalKey#8f02b]
[LabeledGlobalKey<QuillHtmlEditorState>#07cc0]

and QuillHtmlEditor does not have a key property to fix this problem
To Reproduce
Steps to reproduce the behavior
Let's assume that page A has the QuillHtmlEditor

Navigate using Navigator.of(context).push from page A to B
Navigate using Navigator.of(context).push from page B to A
See the error

Expected behavior
Having key property in QuillHtmlEditor to fix this problem
Smartphone (please complete the following information):

Device: [e.g. Galaxy note 9]
OS: [e.g. android 10]

Flutter Doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.3, on Linux Mint 21.1 5.15.0-67-generic, locale
en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.76.1)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
• No issues found!

Hi @HasanAlqaisi,
Could you please share a sample code to understand this better?
The possible reason I see here is that the controller might be defined as a global variable, which is being passed to the new editor page on push ( using same controller for multiple widgets). We need to define the controller inside the page's class so that it will generate new controller for the new screen that is being created on push.
Thanks :)

Hi @HasanAlqaisi,
Could you please share a sample code to understand this better?
The possible reason I see here is that the controller might be defined as a global variable, which is being passed to the new editor page on push ( using same controller for multiple widgets). We need to define the controller inside the page's class so that it will generate new controller for the new screen that is being created on push.
Thanks :)

HI,
You are right. I am using riverpod and the controller provider is not being disposed since there are listeners for it.
So the same controller provided used for multiple widgets
Thanks for helping!

Glad it is resolved.
Thanks for confirming :)


Bug in _removeModels: function(models, options) {

var index = this.indexOf(model); returns always -1
var index = this.models.indexOf(model) returns the right result

@Joc-Ne Thanks for reporting. Could this be a duplicate of #3882 and if so, does upgrading to Backbone 1.4.1 solve the issue for you? Otherwise, please provide a bit more context.


implement GraphQL GET requests

Currently, ANET only implements POST requests. In order to integrate with other systems, it is required to also implement GET requests.

Currently, ANET only implements POST requests. In order to integrate with other systems, it is required to also implement GET requests.
Additionally, the GET request should have an optional query field output.
http://myapi/graphql?query={me{name}}&output=xml
When output equals to json or undefined, the default behavior should be preserved and json returned.
When output equals to xml the query result should be converted to xml with a canonical conversion and returned,

We should be aware that GET requests impose a limitation that POST requests do not have: a maximum length. The exact maximum depends on both the client (browser) and the server, but practically it is in the order of a few KBytes. As long as our queries are relatively short & simple, GET is fine, but keep in mind that for longer queries (like in the future searching for a lot of tags perhaps) it may become necessary to use POST.


Uncaught Error: socket hang up

[Enter steps to reproduce below:]

having unstable internet in the train :)
was checking my mail, when i got back it showed the error.

Atom Version: 0.186.0
System: Mac OS X 10.10.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: socket hang up
At events.js:94

Error: socket hang up
  at createHangUpError (_http_client.js:216:15)
  at TLSSocket.socketOnEnd (_http_client.js:294:23)
  at TLSSocket.emit (events.js:138:20)
  at _stream_readable.js:909:16
  at process._tickCallback (node.js:364:11)

Commands
     -1:16.6 core:confirm (atom-text-editor.editor.mini)
     -1:02.7 core:backspace (atom-text-editor.editor.mini)
     -0:59.1 core:move-right (atom-text-editor.editor.mini)
     -0:58.4 core:backspace (atom-text-editor.editor.mini)
     -0:58.0 core:confirm (atom-text-editor.editor.mini)
     -0:49.5 core:copy (atom-text-editor.editor)
     -0:45.4 core:paste (atom-text-editor.editor)
     -0:36.9 core:save (atom-text-editor.editor)

Config
{
  "core": {},
  "editor": {
    "fontSize": 13,
    "showIndentGuide": true,
    "autoIndent": false,
    "autoIndentOnPaste": false,
    "invisibles": {}
  }
}

Installed Packages
# User
Sublime-Style-Column-Selection, v1.2.3
atom-beautify, v0.21.2
atom-lint, v0.20.1
atom-spotify, v1.2.0
autocomplete-paths, v1.0.1
autocomplete-snippets, v1.0.0
circle-ci, v0.3.0
color-picker, v1.4.4
editorconfig, v0.3.0
emmet, v2.3.2
file-icons, v1.4.8
language-diff, v0.2.0
language-jade, v0.3.0
language-latex, v0.4.1
language-matlab, v0.1.0
language-nginx, v0.4.0
language-rspec, v0.3.0
language-rust, v0.3.5
language-stylus, v0.1.0
terminal-status, v1.3.5
test-status, v0.27.1
travis-ci-status, v0.11.2

# Dev
No dev packages

@kevinsawicki Looking at https://github.com/atom/atom/issues/5035#issuecomment-69956737, it seems that this should have been fixed in a version older than reported in this issue. Any chance there's another place this needs to be addressed or that this is caused by a package somehow?

I'm pretty sure was fixed by https://github.com/atom/notifications/pull/62 and released


Delete files from cache

Is there a way to (automatically) remove all the created thumbnails in the cache folder? I did try the delete function from sorl itself but that didn't work.
I'm using this function
@receiver(post_delete, sender=Image)
def submission_delete(sender, instance, **kwargs):
    instance.image.delete(False)

to delete the original file when deleting my model, but the thumbnail-files in the cache will not be removed by doing this.

I guess that's outside the scope of sorl-thumbnail-serializer-field. As it just uses sorl-thumbnail to generate image and cache them, storage and persistence would probably be controlled using sor-thumbnail's cache settings, and possibly some Django cache backend settings.
It sounds a bit like this is sort of what you already tried out, but I'd assume sorl-thumbnail management commands would remove cached images, and if not there's probably something wrong with the cache setup.
With that said, I do very little Django development these days, and has not been using sorl-thumbnail-serializer-field in a few years myself, so things may very well have changed since this code was written 3-4 years ago.

Thank you very much for your answer. I digged a little deeper into the sorl-thumbnail documentation and I did indeed find a settings parameter that would let me decide how long to store files in the cache.

If anyone stumbles upon this issue in the future, this is how I could solve it:
@receiver(post_delete, sender=Image)
def submission_delete(sender, instance, **kwargs):
    sorl.thumbnail.delete(instance.image, delete_file=True)


Move inline JS to external requests

To increase the time to first paint of the page, we should aim to reduce the HTML page size. One step to doing that is removing the inlined JS. (Part of https://[GHE]/CFGOV/platform/issues/1824).
Removals

banner-footer-webpack-plugin plugin.

Changes

Removes inlined JS and moves it instead to external JS asset requests.

Testing

./frontend.sh and check

http://localhost:8000/about-us/careers/current-openings/ for testing template-level script (single.js)
http://localhost:8000/ask-cfpb/ for testing page-level script (index.js)
http://localhost:8000/about-us/the-bureau/bureau-structure/ for testing on-demand script (on-demand/bureau-structure.js)
**Note: I don't think you need ./cfgov/manage.py collectstatic, but try that if the scripts 404.
Screenshots
HTML byte size before:

HTML byte size after:

Highly recommend an improvement to this PR to bundle all the component JS into one single file and have that loaded by all pages. This is an MVP improvement over the status quo, in which every page has to re-download and parse the inline JavaScript, without the benefits of caching the JS or loading non-JS-dependent content more quickly.
Subsequent improvements to bundling the JS files can still be implemented and tested later; but for now we want to see the first meaningful paint time decrease, and serving the JS as a separate single file will accomplish that.

@anselmbradford what page are those graphs for? In theory the gains for a page like blog posts would be minimal whereas one with a ton of scripts would be huge. I think your best bet would be to compare a few pages with a control version (aka live now), your individual component files, and @cfarm's suggestion for one component file. There just doesn't seem to be enough information here to determine the best course of action.

Another thought is to A/B test our different bundling options to see what the impact is on Google Analytics metrics for mobile users we are tracking: Avg. Document Interactive Time and Avg Page Load Time.

@anselmbradford, for now,  just enable the compression and get a BEWD to sign-off. We can work on the generating various bundles.

@sebworks would that be your work here https://github.com/cfpb/cfgov-refresh/pull/1409, what shouldn't I take from there? E.g. why was it closed?

what page are those graphs for?

It's the homepage.

It's the homepage.
Are we even loading many individual components there? Do we have a page with multiple scripted components, like maybe a video player and expandables or expandables and sortable tables?

@sebworks would that be your work here, what shouldn't I take from there? E.g. why was it closed?

I closed it because I wasn't sure if having an external script was better. If you goto Google.com or Facebook.com all their scripts are inline. Let's just test out using the external scripts and see how they perform.

Can I get a bewd signoff @rosskarchner @willbarton @richaagarwal @chosak ?

For posterity, the possible JS bundling options @cfarm @sebworks and I explored 11/2:


Supported subtitles formats

I would like to know which subtitle formats are accepted by Clappr.
I've made it work with .vtt format. I also tried sub and srt, but it didn't work. Do I have to use only .vtt format ?
In order to make it work I used playback.externalTracks option like this:
playback: {
	externalTracks: [
		{ lang: 'en', label: 'Test1', src: '<host>/assets/subtitles/test1.vtt', kind: 'subtitles', default: true },
		{ lang: 'en', label: 'Test2', src: '<host>/assets/subtitles/test2.srt', kind: 'subtitles', default: false },
		{ lang: 'en', label: 'Test3', src: '<host>/assets/subtitles/test3.sub', kind: 'subtitles', default: false }
	]
}

In this test, all 3 options appeared on cc icon, but only .vtt really worked.
I see there's a srt plugin available: https://github.com/jmvtechnology/Clappr-Subtitle
But as stated in this page: "This is a very early version. You can't select subtitle tracks, and long srt files may run slow."
I can't afford to use an early version as my project will be used by thousands of people enterprise-wide.
There other options I'm missing ?

I think it uses browser video element capabilities. So if subtitle format is supported by browser, it should work.

As stated here there is .vtt support only.
Are there any other way ? In the negative case, are there any plans to incorporate support for other subtitles format ?

I don't know about other format. It seems that major browsers tend to support the vtt format.

Hi @nelson777, if you want to use other formats apart from .vtt I suggest you create subtitle plugins to accept these formats. If you need help creating the plugin send us an issue to ask for help.


Best practice for venv and git version control

This has been addressed in https://github.com/NICD-UK/project-template/pull/18


Fixed waves example CMakeLists.txt

Signed-off-by: ahcorde<EMAIL_ADDRESS>🦟 Bug fix
Summary
Waves example depends on ign-common4 but it was not included.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

is it only the waves example that does not build for you? The other examples also depend common4, e.g. actor_animation, but we also don't specify it in CMakeLists.txt

is it only the waves example that does not build for you? The other examples also depend common4, e.g. actor_animation, but we also don't specify it in CMakeLists.txt

I will open a PR for the other ones


Reject with an Error on timeout

In my previous PR I neglected to throw an Error when I called reject(). Not the end of the world, but was bad form.

Good suggestion. Perhaps we should make named errors for every zero argument reject() in the code?


User detail - Unable to copy the email address

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open App and login
Navigate to any chat tap on photo
Tap on copy icon

Expected Result:
User is able to copy the email address
Actual Result:
User is not able to copy the email address, clicking on it does not turn green
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Version Number: v1.1.89-0
Reproducible in staging?:  Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/186425027-367f83a2-d6ad-4adc-b27d-30a5580dc9c7.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Yes this is broken, and it is a deploy blocker -> easy to reproduce, and can't reproduce in Prod but can in Staging

Depending on how we get on, I'd be fine with deploying with this issue and fixing in the next release.

ok, there is still a workaround and it's not that hard, so that makes sense

Yeah, I agree that I wouldn't hold the deploy on this issue. Should we remove the DB label then?

yeah let's

Fix incoming

@kavimuru oh i see this issue has been fixed and deployed on staging and prod. Am i eligible for reporting bonus ?

@Expensify/contributor-management - Can someone pay @aneequeahmad for reporting the issue? Thanks

I'm on it.
https://www.upwork.com/jobs/~016dc48f3d2b89154f
@aneequeahmad , please apply here and confirm in a post on this issue once you have. Thx

Reopen so I don't forget to pay :)

@mallenexpensify  I have applied, Thanks for mentioning.

Hired @aneequeahmad , let me know when you've accepted and I'll pay.

@mallenexpensify I have accepted the offer.

Paid! Thanks again for the help @aneequeahmad


EPS launcher shows Alien Isolation is "running" when OpenCage is running

I think this is possibly Epic's Launcher at fault here but in case it's not.  I've been running into issues with the launcher throwing Epic's rather generic error 0013 could not launch "verify the game" at various times. All of them linked to Epic's launcher needing to be restarted not a need to reverify.
However, I've noticed that if I have OC open and have tried to launch the game it will also throw this error
even to the point of showing the game as  "running" in the launcher until OC is exited.   It also will not launch the game from the Launch Game option sometimes throwing an error "the game is already running"  but also throwing that error 0013 reverify the game.  Upon exiting OC it immdiately launches.  Or it sometimes requires I close Epic's launcher to clear it out.
This is probably something minor and I rather suspect it's all EPS but in case it's not I've made this issue report.

Nevermind.  I seem to have reported this issue before.  I'm a complete dunce.


Error whiile installation

I was running !pip install ai4bharat-transliteration on google colab and got this error.
Collecting omegaconf<2.1 (from fairseq->ai4bharat-transliteration)
Using cached omegaconf-2.0.6-py3-none-any.whl.metadata (3.0 kB)
WARNING: Ignoring version 2.0.6 of omegaconf since it has invalid metadata:
Requested omegaconf<2.1 from https://files.pythonhosted.org/packages/d0/eb/9d63ce09dd8aa85767c65668d5414958ea29648a0eec80a4a7d311ec2684/omegaconf-2.0.6-py3-none-any.whl (from fairseq->ai4bharat-transliteration) has invalid metadata: .* suffix can only be used with == or != operators
PyYAML (>=5.1.)
~~~~~~^
Please use pip<24.1 if you need to use this version.
Using cached omegaconf-2.0.5-py3-none-any.whl.metadata (3.0 kB)
WARNING: Ignoring version 2.0.5 of omegaconf since it has invalid metadata:
Requested omegaconf<2.1 from https://files.pythonhosted.org/packages/e5/f6/043b6d255dd6fbf2025110cea35b87f4c5100a181681d8eab496269f0d5b/omegaconf-2.0.5-py3-none-any.whl (from fairseq->ai4bharat-transliteration) has invalid metadata: . suffix can only be used with == or != operators
PyYAML (>=5.1.*)
~~~~~~^
Please use pip<24.1 if you need to use this version.
INFO: pip is looking at multiple versions of hydra-core to determine which version is compatible with other requirements. This could take a while.
Collecting fairseq (from ai4bharat-transliteration)
Using cached fairseq-0.12.1.tar.gz (9.6 MB)
Installing build dependencies ... done
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.

You can use a lower version of pip to avoid this error. For example you can do
pip install pip==23.2.1
before doing :
pip install ai4bharat-transliteration


Feature Request: valueLiveUpdate for sap.m.MaskInput

Hi,
are there plans to add valueLiveUpdate to sap.m.MaskIput?
Best regards.

Hello @daletman,
There is liveChange event in sap.m.MaskInput available since SAPUI5 version 1.104: https://sapui5.hana.ondemand.com/#/api/sap.m.MaskInput%23events/liveChange
If it does not fulfil your requirements, please describe in details the use case and why do you need valueLiveUpdate event.
Thank you in advance!
Kind Regards,
Niya

Hello @niyap,
yes there is the liveChange event, but sap.m.MaskInput does not have the valueLiveUpdate property like for example the sap.m.Input. From the property describtion of sap.m.Input which has an valueLiveUpdate property:

Indicates when the value gets updated with the user changes: At each keystroke (true) or first when the user presses enter or tabs out (false).

Note: When set to true and the value of the Input control is bound to a model, the change event becomes obsolete and will not be fired, as the value in the model will be updated each time the user provides input. In such cases, subscription to the liveChange event is more appropriate, as it corresponds to the way the underlying model gets updated.

https://openui5.hana.ondemand.com/api/sap.m.Input#controlProperties
So the functionality I need is, that with ervery keytroke the value in the bound Model is updated, so that an getValue() on the MaskInput does give me the latest value.

Hello @daletman,
Thank you for sharing your enhancement proposal. I've created an internal incident<PHONE_NUMBER>.
The status of the issue will be updated here in GitHub.
Regards,
Niya

Hello @daletman ,
There is JIRA BLI created for this feature request (BGSOFUIBALKAN-7793), and it will be developed in the future.
Best Regards,
Nikolay Hristov


”Coronini” is listed in the drop-down menu

Description:
"Coronini" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Coronini" in the search bar.

Expected results
The location is listed in the drop-down menu as "Coronini, Caraș-Severin".


Figure something out for horses

Biggest design issue for the update to 1.6. Suggestions wanted.
Issues with horses:

Low cost, very fast travel - needs to have some cost (maybe riding drains hunger?)
Taming process is clunky
Attribute system is opaque - should it be removed and standardized, or should it be made more visible in game? Could create interesting side project like sheep colors, but needs to be communicated better if it is maintained
General animageddon updates

Horses -- the squid of the plains.
On a serious note, saddles needing a good amount of tanned leather might stop early game mischief.  Draining hunger also seems good.  That moves them to later game or emergency use.

Making them difficult to tame could also be an idea. And drastically draining player hunger would not make much sense from a logical standpoint at least, so I guess instead/additionally draining horse hunger could also be insteresting.

Horse hunger sounds great - they could refill it slowly by grazing like cows/sheep do, or be fed with hay/carrots/tall grass to speed up the process.
As an alternative to xp cost, taming could also be a one-time player hunger drain, kinda like horses in Zelda BOTW/TOTK drain stamina when first mounted until they calm down. Both options seem reasonable, but using xp feels a bit off considering how xp is used for other purposes.

I think there are a few main issues to solve here:
1.  When is the player meant to have access to RIDING horses?
In my opinion they would fit nicely just before player goes out to explore for villages and the gating would be by having
saddles be crafted from something that is made on a stoked cauldron or something.
2.  When is the player meant to have access to TAMING horses?
I think they should be tameable from as soon as player has spare food, but that comes with the cost of maintaining/caring them.
3.  How is the player going to tame horses?
By continuously maintaining/caring for them over multiple days, doing so would raise a taming value which eventually improves a horse's value. (by generating power faster, moving faster, increasing horse's willingness to follow the player)
4.  What is the advantage that horses are meant to give to players for their efforts?
Since I proposed they should be tameable from day 0, I also believe they must have some value other than faster travelling through riding, and as such I can think of two things:

Horses can carry containers, much like Llamas on modern mc, making them a mobile chest.
Before player is able to ride horses they can only pull them either with a lead or by alluring them with food ( both of which should be slower than riding ) and afterwards when they can ride them they would become reasonably mobile chests.
Horses can generate (slower than wind mill) mechanical power through a Horse Mill.
This would add a way of generating mech power underground.

For both of these ideas, I don't think horses should spend more food than it does just by being around, as players would just rather not do it since having spare food early/mid game is harder.
5.  How are horses meant to be maintained/cared for?
Food.
These ideas have not been validated to be balanced, but I still wanted to get them out there.

Moving this out of the 1.6 update epic as riding horses will likely not make it in to initial release. Other changes have been split into their own tickets


Docs: explain that global SCSS should be imported the vite way

For example, as a file import statement in the entry JS for a page:
// index.js
import './path/to/styles/style.scss';

Note: path should be relative to .cayo/.

Closing in favor of #14


Make navbar more mobile friendly

Motivation
At the moment, the website's navbar has a variety of UX problems on mobile, including sizing, overflow, and other issues.
Modifications
I've removed some CSS styling that either wasn't necessary or created a broken experience.
Result
The navbar will provide an experience more in line with what users will expect. Not perfect but I think a major improvement.

retest this please

retest this please

@merlimat I'm getting this (non-website-related) error in CI:
[ERROR] Tests run: 542, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1,225.036 s <<< FAILURE! - in TestSuite
[ERROR] socketTest(org.apache.pulsar.websocket.proxy.ProxyPublishConsumeTest)  Time elapsed: 0.198 s  <<< FAILURE!
java.lang.AssertionError: null: lists don't have the same size expected [6] but found [10]
	at org.apache.pulsar.websocket.proxy.ProxyPublishConsumeTest.socketTest(ProxyPublishConsumeTest.java:160)

[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   ProxyPublishConsumeTest.socketTest:160 null: lists don't have the same size expected [6] but found [10]
[INFO]
[ERROR] Tests run: 542, Failures: 1, Errors: 0, Skipped: 0
[INFO]
[ERROR] There are test failures.

Should I submit an issue?

Should I submit an issue?
There's already an issue open for that flaky test, though is not easy to replicate in local env.

retest this please


haskellPackages: misc fixes for darwin

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

These are quite unimportant leaf packages and I probably should've checked before investing the time to fix them. More careful testing is probably not necessary, they seem to build.


FLV Video stream not displaying

I am pulling in a FLV stream from my Security DVR. I cannot change the output from FLV. My computer shows that a 1.5Mb stream is being used for 10 sec then stops at no time did the MediaElement show the video though the CurrentStateChange Event says its playing.
I can play a FLV file fine but not in a continuous stream.
When playing in VideoLan (VLC) the stream info is
H264-MPEG-4-AVC part 10 (avc1)
decode: Planar-4.2.0 YUV
I believe the problem resides in the fact that when adding the libx264 to the FFMpegConfig.sh file for the architecture FFMpeg errors on compile that i am missing x264.h Am I using the wrong option in the FFMpegConfig.sh to have ffmpeg compiled with x264 support?

Hi @mscott161
Any chance you could share a sample file that exhibits the issue?
Thanks

Hi @khouzam ,
The flv stream is coming from a DVR.  There is not file.  The FFMpegInterop will play saved files just fine, but when receiving the live stream in the same format it does not show.  I provided the details of the stream I got from VLC in the initial question.


OpenAPI document missing the security scheme definition

According to the Authorization section of the API document, to make an API call it requires one mandatory Authorization header and an optional OpenAI-Organization header.
As both are missing in the OpenAPI document, they need to be added. Here are examples:
...
paths:
  ...
  /models:
    get:
      operationId: listModels
      tags:
        - OpenAI
      summary: Lists the currently available models, and provides basic information about each one such as the owner and availability.
      # add optional header parameter like below
      parameters:
        - in: header
          name: OpenAI-Organization
          required: false
          schema:
            type: string
            example:
              org-hP479gBqdEuqjgkVyFPZ5g5h
          description: For users who belong to multiple organizations, you can pass a header to specify which organization is used for an API request. Usage from these API requests will count against the specified organization's subscription quota.
...
# global authorization
security:
  - api_key: []

components:
  # authorization scheme
  securitySchemes:
    api_key:
      type: 'http'
      scheme: 'bearer'
      bearerFormat: 'Bearer'
...

Thanks @justinyoo, I would also like to see these two things added.
#12 partially solves this (Adds the AuthorizationsecurityScheme).
Adding the OpenAI-Organization Header Parameter to all endpoints would also be great.

@om4james I've created a PR for this issue at #26 which has broader coverage than the other two #10 and #12

Was about to ask the same thing, good to see someone already thought about it. Can't wait for it to be merged

Security schema is now fixed, the org ID will take a bit more work but is on my radar! Thanks again for flagging.


Add ForwardContext to wrap model forward pass

Depends on #257, #263

This PR adds a new ForwardContext class which provides a thread-local context to the forward pass of a model. This context can be used to store information relevant to the current forward pass without the need to add new parameters to the method definitions.
The context should be added to all adapter-supporting models by decorating the respective base model forward() methods with @ForwardContext.wrap.
This functionality is mainly for future use. Currently, it only replaces the pre_transformer_forward() method.

This PR/issue depends on:

Adapter-Hub/adapter-transformers#257

Automatically provided by Dependent Issues (🤖).

This PR/issue depends on:

Adapter-Hub/adapter-transformers#257
Adapter-Hub/adapter-transformers#263

Automatically provided by Dependent Issues (🤖).


Minify option is broken when used with reactify plugin

Says SyntaxError: Unexpected token punc «(», expected punc «:» while parsing file

My bad, I had wrong mix of ES5 and ES6 and I didn't realize that until I ran uglifyJS manually.


Can't use on Windows

I'm trying to compile empty bin project with this crate as dependency on Windows. I'm getting such error:
error: failed to run custom build command for `librocksdb-sys v5.7.1`
process didn't exit successfully: `C:\Users\вася\Projects\test1\target\debug\build\librocksdb-sys-d4cbcc9a730ac9e2\build-script-build` (exit code: 101)

--- stdout
cargo:rerun-if-changed=build.rs
cargo:rerun-if-changed=rocksdb/

--- stderr
thread 'main' panicked at 'Unable to find libclang: "couldn\'t find any of [\'clang.dll\', \'libclang.dll\'], set the LIBCLANG_PATH environment variabl
e to a path where one of these files can be found (skipped: [])"', src\libcore\result.rs:906:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.

append  port/win/win_thread.cc  to rust-rocksdb/librocksdb-sys/rocksdb_lib_sources.txt  like:

see #164

A temporary fix to this is to install LLVM for Windows, I used version 5.0.1.
http://releases.llvm.org/download.html

This should be closed.

It seems that version 0.21 still has this problem.


Really fix long strings with ODBC

This also works with the old ODBC drivers.
See #383.

Merging this because the CI failure is bogus, it's due to failing to download Oracle RPM.


Default build command not working with recent AEM SDK

Expected Behaviour
Default build command / AEM packages deployment working correctly.
Actual Behaviour

The second package installation fails ("ui.content") as AEM is not responsive. Suspecting inactive component PackageManagerServlet at the time of installation. Observed many retries performed by Maven content package plugin when listing packages without effect. After waiting for some time (awaiting stabilization of PackageManagerServlet) second attempt using the same command ends with success.
Also observed many OSGi events being triggered infinitely which suggests general instance instability (observable just after turning on instance / regardless package installation), originally spotted here: https://github.com/Cognifide/gradle-aem-plugin/issues/726
Steps to Reproduce
Set up AEM author and publish instance using "aem-sdk-quickstart-2020.7.3902.20200716T022312Z-200604.jar"
Generate project from archetype snapshot (actual master) using values
mvn -B archetype:generate \
 -D archetypeGroupId=com.adobe.granite.archetypes \
 -D archetypeArtifactId=aem-project-archetype \
 -D archetypeVersion=23 \
 -D aemVersion=cloud \
 -D appTitle="My Site" \
 -D appId="mysite" \
 -D groupId="com.mysite" \
 -D frontendModule=general \
 -D includeExamples=n

Platform and Version
Apache Maven 3.6.3
Java version: 1.8.0
OS name: "mac os x", version: "10.15.4", arch: "x86_64", family: "mac"

@pun-ky what maven command are you running?

@godanny86 Is visible on screenshot.
I was checking documented build command plus deploying also on publish. So I added autoInstallPackagePublish.
Right now I've got even more concerns.

what is most recommended command to build?
is it possible to deploy to author and publish during single Maven execution (to avoid building artifacts twice) or I need to run build twice with different profiles? It would be nice if it will be documented.
when I have run command on screenshot again then the build ended with success, however 'all' package was also deployed on instance. Strange. I am totally sure that I haven't activated that profile that you mentioned. My intention is to have only deployed ui.apps and ui.content.

I am aware that AEM dislikes installing package one after another however documentation here is not telling anything which command / approach is most preferred.

The profiles to build+deploy the entire project (running from the root of the parent pom) would be autoInstallSinglePackage (author) and autoInstallSinglePackagePublish(publish).
If you want to build the modules individually you can navigate into the folder (i.e cd ui.apps) and use the autoInstallPackage / autoInstallPackagePublish profiles.
Agreed that this needs to be reflected in the generated README and in the Docs:

https://github.com/adobe/aem-project-archetype/blob/master/src/main/archetype/README.md
https://docs.adobe.com/content/help/en/experience-manager-core-components/using/developing/archetype/using.html#building-and-installing

Is there other documentation you noticed that needs more information on recommended build commands?

Indeed, the ability to build modules individually only from the subdirectory is a clue here which answers to my problem reported. The second thing is the primary build command to be updated.

Is there other documentation you noticed that needs more information on recommended build commands?

That's all right now. Thanks @godanny86

@godanny86 thank you for the contribution :) merged PR,


Consider removing dependency on ORM

Context
In the main comment thread of https://github.com/navapbc/wic-mt-demo-project-mock-api/pull/20 we discussed not having ORM be in the default template.

upon further thought i think we can ice this for now


Forms

Describe the bug
Can't select Wormadam version (renegade platinum)
To Reproduce
Steps to reproduce the behavior:

Choose Burmy on a route as an encounter (in my case it was route 202)
Click Evolve
Only Plant Cloak Wormadam or Mothim is available.

Expected behavior
Trash Cloak and Sandy Cloak Wormadam should be available as an option here.
Desktop (please complete the following information):

OS: Windows 10 Pro (Build 19045.2728)
Chrome (Version 111.0.5563.147)

Additional context
Looks like based on the source Pokemon.json that the other Wormadam forms aren't in yet, or to put it another way, that forms of the same pokemon (without having a different name) aren't quite implemented yet.

Hi @EmberAnimus ! Thanks for raising this
For more context, I had to take a dump of the Pokemon.json as a library I was using was deprecated.
It missed out alternate forms, but obviously they're needed for things like wormadam!
I've added them back in now, ✌


Set zoom not working in Cataclysm Classic

Hi,
First, thank you for this addon, I use it on retail and classic for quite some time now and I love it.
The issue I have arose with Cataclysm Classic where the Set Zoom option is not working anymore in set situations. I usualy use it when I'm in mounted situation or in combat to have a more distant view and it was working correctly in WOTLK Classic.
As you can see the situation is well recognised, but the zoom level does not change when I enter or exit it. (if I use the /zi function for this screenshot it shows 13.88)

If I use the set view option instead it works correctly but customized views are not what I need here unfortunately. Actually, all other options seems to be working fine.
Edit 1 : Also, if I do a /reload while mounted, it then zooms out correctly to the set distance, but it doesn't change anymore afterward.
Edit 2 : If I zoom out while on foot then mount, it goes to the set view of the parent situation (in this case I set the "world" view to standard view 5, so it goes back to that standard view if I mount).
Edit 3 : Removing any parent situation allow the "set zoom" option to act correctly (I actually have the "world" situation activated with "set view" on) -> Maybe a priority issue between the two situations ?
Thank you again !

Hello and thanks for your kind words. :-) Also thanks for your elaborate description of the issue. I just tried to reproduce what you are experiencing. But I find the "Set Zoom" to work as expected when I change situations.
Have you tried using "Zoom Type" "Set" instead of "Out"? Does it still not work for you then?
Have you also tried running DynamicCam as your only enabled addon to make sure there is not other addon interfering?
If you have checked all of that, maybe you want to send me your WTF\Account<account name>\SavedVariables\DynamicCam.lua file. So I can replicated your exact settings. Just post the content here or send it to<EMAIL_ADDRESS>Looking forward to hearing from you again.

Hello,
I did try different settings for the Zoom Type and also only activating the DynamicCam addon but it didn't change a thing.
You'll find the lua file attached, hopefully you'll find something to work it out.
DynamicCam.zip

Hello, and thanks for your configuration file. I could now see what your issue is.
In the logic of DynamicCam a "set view" always overrides a "set zoom". And "set view" also includes "restore view". In your parent situations you have "set view" with "restore" enabled. So whenever, you leave these situations the saved view of when you entered them is restored, which overrides a possible "set zoom" of the other situation you are entering.
It could be programmed differently, such that the "set zoom" of an entered situation overrides (or maybe even better follows) the "restore zoom" of the previous situation. I will consider it.
For the time being, it would probably make most sense for you to disable "restore view" for your parent situations.
That you just noticed this problem with the launch of Cata classic must have been a coincidence, because it has always been like this.
Thanks again for getting in touch! Let me know if you need anything else.


fix: process entry rune-by-rune without casting between string -> rune -> string

Converting between types is costly and can waste a lot of memory. Go lets us iterate through strings rune-by-rune, which helps to reduce some conversions.

cc @wadearnold

Codecov Report

Merging #1305 (baed52a) into master (c35e195) will increase coverage by 0.03%.
The diff coverage is 97.05%.

:exclamation: Current head baed52a differs from pull request most recent head 6da4042. Consider uploading reports for the commit 6da4042 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1305      +/-   ##
==========================================
+ Coverage   88.48%   88.52%   +0.03%
==========================================
  Files          73       73
  Lines        7089     7111      +22
==========================================
+ Hits         6273     6295      +22
  Misses        480      480
  Partials      336      336


Tokio 1.0

Closes #21
The tests currently fail because of an executor mismatch with reqwest. We can either wait for it to be updated, or rewrite the tests to use the hyper client.

Thank you!

The tests currently fail because of an executor mismatch with reqwest. We can either wait for it to be updated, or rewrite the tests to use the hyper client.

I think there's some interoperability testing benefit to using reqwest. If you're happy waiting, I think it's probably better to keep that. I see there's a reqwest issue and PR for the upgrade, and seanmonstar said it's coming soon.

Tests notwithstanding, I can confirm that http_serve::serve works with hyper 0.14.

Looks like the new reqwest is released. :-)

Looks like the new reqwest is released. :-)

Updated.

Updated.

CI passes now, too (finally).

CI passes now, too (finally).

Can you also publish a new version when you find time? Thanks.

Can you also publish a new version when you find time? Thanks.

Published, and thanks!

Published, and thanks!


Transparency for "Isosurfaces"

I want to set transparency for "Isosurfaces".   Color can be changed by “transfer function” but not transparency.

looking into it

FYI, transparency for Isosurfaces is now working in devel branch. Transparency is controlled via the material that is assigned to the geometry, see for example the StructuredVolume tutorial.


Foreign key value is null

Hi!
I have a problem:
When I create ManyToOne relationship by @ManyToOne decotator, I want to select foreign key value.
But on field select result is null.
How can i get a foreign key value?
I`m tryng to use something like this:
@Field(() => User)
@ManyToOne(() => User, { nullable: true })
user?: User;
@Field()
@Property({ hidden: false, fieldName: "user_id", unsigned: true })
userId: number; // result is NULL but entity exist (in db FK has value) and other fields has values
Thanks for help!

You should not define the other property (userId), work with the user instead. Just read ent.user.id, it will be there even if you do not populate the relation.

This behavior by design? Okey, thaks for a good orm =)

Yes, this is by design.

You should not define the other property (userId), work with the user instead. Just read ent.user.id, it will be there even if you do not populate the relation.

But what if i want to include a foreign key too?

Every database column in the entity is mapped to one property. There is no need for the other column, it just adds clutter, also brings inconsistency (you would the have multiple places where you could define the column name, type, length, etc - which one should work then?).
If you do not like that behaviour, it should be possible to implement what you want via custom hydrator.

Maybe show me the use case so I can advice better.

Primarily a backwards compatability with output from our previous orm.
There are many places in our app, which use foreign key directly.

If it is only about being able to read values from existing entities, you could use @OnInit hook to fill that property.
But you should define it as persist: false so it is ignored in the schema generator.
Although, if you have consistent naming (prop and propId), you could use simple find/replace to change propId to prop.id, right?
Or as I mentioned, you can implement custom hydrator. I can help with that, but not right now.
https://github.com/mikro-orm/mikro-orm/blob/master/lib/hydration/ObjectHydrator.ts
https://mikro-orm.io/docs/configuration/#custom-hydrator

Actually there is quite easy way to achieve this, not sure why it did not pop up on my mind before. You can define the userId property as getter that will read the value from user.id:
@ManyToOne(() => User, { nullable: true })
user?: User;

@Property({ persist: false })
get userId(): number {
  return this.user.id;
}

This is called virtual property in MikroORM: https://mikro-orm.io/docs/defining-entities#virtual-properties
If you want to be able to set to that property, it is a bit more tricky, as you need to set instance of the target entity to the user variable. It can be done via internal entity manager reference if the parent entity is loaded from database (and therefore managed).
@ManyToOne(() => User, { nullable: true })
user?: User;

@Property({ persist: false })
get userId(): number {
  return this.user.id;
}

set userId(value: number): void {
  this.user = wrap(this).__em!.getReference(User, value); // TODO __em can be undefined
}

Thanks for detailed answers, really appreciate that!


Running on Azure VM works smoothly

Got everything up and running on Azure VM.
Comments:

Download of images is way faster, probably due to hosting of images on Azure Container Registry (I guess).
Extraction of image files sometimes hangs, but after closing PowerShell and checking the images the downloaded image is available and can be started. This happens with both NAV images and Windows Server Core image.
VM with SSD drive seems to noticeably faster when it comes to image download / extraction and starting the container
Starting the container is faster compared to Windows 10 and has less memory consumption
Publishing ports requires to add an inbound security role on Azure

Now I'm going to test with multiple containers and trying to initialize a classroom setting.
Nice work!

Cool!!!
About the performance comparison:

On Win2016 you are running by default EE version of Docker which (again, by default) uses process isolation  (On Win10 you can use only hyper-v isolation, on W2K16 you can choose for each container but there some licensing restrictions when using hyper-v isolation on the server).

process isolation performs much better as this is the native way you can see on Linux. You can see processes using the task manager - in the case of nav-docker you will see running SQL and NAV server as well as IIS. And this will be multiplicated by containers you are running on the host.

hyper-v isolation needs to create very lightweight VM to run a container. But still, there is a VM in between. This approach is better in case you need to achieve higher level of the isolation (for any reason you can have) but, and this is obvious, you need to consider performance penalizations.

Thanks Koubek. Didn't expect process isolation to be such a big difference. Don't see the separate processes though.
Any idea what memory consumption would be when running up to e.g. 25 nav containers on one single VM?
What about CPU requirements for such set up?
I compared prices for big Azure machines, the larger machines will be cheaper on a daily basis compared to running multiple small VM's.

This is what we have on one of the docker servers (there are 2016 and 2017 instances and 2 or 3 CUs running inside):

The problem of the memory actually is that NAV increases the consumption when starting compiling the objects and later based on the demand. So it is pretty unpredictable. Anyway, on W2K16 you don`t have to set the memory limit to solve the problem we can see on Win10. The memory would be consumed dynamically.
I expect sth. about 850 MB - 1,2 GB per instance but this could vary.
CPU should be optimized better than memory. But again, when compiling the CPU consumption goes pretty high and can use multiple cores.
You can run multiple instances and start using them and then run docker stats and watch the results.
One more detail - we in the company use gMSA and then I have three SQL instances (different versions) and the NAV containers run only NAV + IIS (with NAV Web). When running without SQL inside the performance is much better in our case. So whenever possible I would recommend using NAV like we do. Of course, I can understand that for classes would be much better to use all-in-one solution. Moreover you do not depend on gMSA.

I would estimate the NAV memory and CPU consumpition of X container instances almost exactly like the consumption you'd calculate for X NAV instances in one machine because there is a close to 0 overhead for the containers as possible

I'm talking about trainings for VS Code, so compiling and deployment is different anyway. Would expect much less impact on the server. Anyway, it seems to be safe to reserve about 2 GB per container. Depending on the number of trainees, we can choose the VM size.
Alternatively, we choose to host the databases on Azure SQL. If we prepare one image with the connection settings, then we only have to change the database name when starting the container. I guess it would save about 500 MB memory per container.
But first step is to find out how we can deploy Extension packages from local VS Code the docker container in Azure VM. Authentication is the challenge here (maybe that's already solved, need to find out). If that works, then it's a matter of preparing images and off we go...

You can use UserName authentication. You'd then need to expose 7049 (and 46/47/48, depending on what you need) and 80/443 on the host and map those through Azure admin to make them available externally. If you want everyone to be able to download the .vsix, you'd need to expose 8080 as well. I am wondering if the triple hostname challenge (azure external name vs. azure internal name vs. container internal name) will cause problem. This https://azure.microsoft.com/en-us/blog/announcing-azure-container-instances/ would make it even easier, but I haven't see anything about Windows support there

That's what I'm playing with right now. And succeeded to download AL package from Azure VM to local VS Code.

Publish all ports on the container (with -P). No harm is done since those ports are only available inside the Azure VM and not from outside. With -P you get automatic port mapping with random port numbers.
Added the generated port numbers for port 8080, 443 and 7049 to Azure security rules.

Now I was able to download vsix, certificate and open the Web Client. With a fully qualified domain name, like https://mydocker.westeurope.cloudapp.azure.com:10394/NAV/WebClient/. Login works, so far so good. (BTW, it's FAST !)
Next steps:

Install downloaded vsix in Visual Studio
Initialize a new workspace
Change these settings in launch.json:

"server": "https://mydocker.westeurope.cloudapp.azure.com:10394",
"serverInstance": "NAV",
"port": 36152,
"authentication": "UserPassword",

Change locale to W1 in app.json
Execute command AL:Download symbols

The last step asked for username and password and then nicely downloaded the symbols.
It's very promising...
Next step is to close the loop and publish an extension (expect no problems).

Just published the Hello world example to docker container on Azure VM.
The publish process worked without problem. Credentials were taken from the credential cache, didn't have to re-enter them.
The launch process had some problems.

Launch didn't respect the port number in the server setting. See setting in previous post, it has port number 10394. But the composed url in the browser had no port number at all (and used 443 as a result). That will be a problem with multiple containers running on the same VM
The composed url had a debugging context parameter. But then the web client failed with an error message 'Sorry, that didn't work.' The container window showed an event message, which I will post below in a separate message.
Chrome has a problem with the generated SSL certificate. I have imported the certificate into the trusted root certificates and Edge is happy with that. But Chrome still throws an error: This server could not prove that it is mydocker.westeurope.cloudapp.azure.com; its security certificate does not specify Subject Alternative Names. This may be caused by a misconfiguration or an attacker intercepting your connection.

First two issues should probably also go to the AL project issues. @freddydk, what do you think?
Having said that... Removing the debuggingcontext parameter from the url, adding the correct port number and using Edge, I got the Hello World extension working! Isn't that great?

Event viewer message in container related to debuggingcontext parameter:
TimeGenerated : 8/14/2017 9:19:05 PM
EntryType     : Warning
Message       : Server instance: NAV
                Category: Runtime
                ClientSessionId: 00000000-0000-0000-0000-000000000000
                ClientActivityId: 00000000-0000-0000-0000-000000000000
                ServerSessionUniqueId: 408b4b08-f84a-4ccb-aeb3-9ebb416fead7
                ServerActivityId: 8c19bc1f-46b7-45e3-a1aa-83fa8a4eff00
                EventTime: 08/14/2017 21:19:05
                Message <ii>(InvalidOperationException): <ii>Dynamic
                operations can only be performed in homogenous AppDomain.
                ExceptionStackTrace:
                   at System.Runtime.CompilerServices.CallSiteBinder.BindCore[T
                ](CallSite`1 site, Object[] args)
                   at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TR
                et](CallSite site, T0 arg0, T1 arg1)
                   at Microsoft.Dynamics.Nav.Service.Dev.Web.Controllers.Debugg
                erHub.OnDebuggeeConnected(Object sender,
                DebugSessionStartEventArgs eventArgs)
                   at Microsoft.Dynamics.Nav.Runtime.Debugger.DebugRuntimeManag
                er.OnDebuggingSessionStarted(DebugSessionStartEventArgs e)
                   at Microsoft.Dynamics.Nav.Runtime.NavSession.StartDebuggingS
                ession(String debuggingContext)
                   at Microsoft.Dynamics.Nav.Service.NSService.OpenConnection(C
                onnectionRequest connectionRequest)
                   at SyncInvokeOpenConnection(Object , Object[] , Object[] )
                   at
                System.ServiceModel.Dispatcher.SyncMethodInvoker.Invoke(Object
                instance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Er
                rorMappingCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, MethodBase syncMethod, Object[]
                inputs, Object[]& outputs)
                CallerStackTrace:
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Er
                rorMappingCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, MethodBase syncMethod, Object[]
                inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass24_0.<Combine>b__1(NSServiceBase
                serviceInstance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Pu
                shPopCombinator(ServiceOperation innerOperation, NSServiceBase
                serviceInstance, MethodBase syncMethod, Object[] inputs,
                Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass24_0.<Combine>b__1(NSServiceBase
                serviceInstance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Cr
                eateNewSessionCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, Boolean requireNavUser,
                Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Cr
                eateNewAdminSessionCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, MethodBase syncMethod, Object[]
                inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass24_0.<Combine>b__1(NSServiceBase
                serviceInstance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass10_0.<PerformanceCounterCombinator>b__0()
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Pe
                rformanceCounterCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, MethodBase syncMethod, Object[]
                inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass24_0.<Combine>b__1(NSServiceBase
                serviceInstance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.Tl
                sClearCombinator(ServiceOperation innerOperation,
                NSServiceBase serviceInstance, MethodBase syncMethod, Object[]
                inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.<>
                c__DisplayClass24_0.<Combine>b__1(NSServiceBase
                serviceInstance, Object[] inputs, Object[]& outputs)
                   at Microsoft.Dynamics.Nav.Service.ServiceOperationInvoker.In
                voke(Object instance, Object[] inputs, Object[]& outputs)
                   at System.ServiceModel.Dispatcher.DispatchOperationRuntime.I
                nvokeBegin(MessageRpc& rpc)
                   at System.ServiceModel.Dispatcher.ImmutableDispatchRuntime.P
                rocessMessage5(MessageRpc& rpc)
                   at
                System.ServiceModel.Dispatcher.MessageRpc.Process(Boolean
                isOperationContextSet)
                   at System.ServiceModel.Dispatcher.MessageRpc.Wrapper.Resume(
                Boolean& alreadyResumedNoLock)
                   at System.ServiceModel.Dispatcher.ThreadBehavior.ResumeProce
                ssing(IResumeMessageRpc resume)
                   at Microsoft.Dynamics.Nav.Runtime.NavSynchronizationContext.
                <>c__DisplayClass1_0.<ClearThreadLocalStorageDelegate>b__0(Obje
                ct state)
                   at
                System.Threading.ExecutionContext.RunInternal(ExecutionContext
                executionContext, ContextCallback callback, Object state,
                Boolean preserveSyncCtx)
                   at System.Threading.ExecutionContext.Run(ExecutionContext
                executionContext, ContextCallback callback, Object state,
                Boolean preserveSyncCtx)
                   at System.Threading.QueueUserWorkItemCallback.System.Threadi
                ng.IThreadPoolWorkItem.ExecuteWorkItem()
                   at System.Threading.ThreadPoolWorkQueue.Dispatch()

                </ii></ii>
                ProcessId: 356
                Tag: 00000HC
                ThreadId: 316
                CounterInformation:

And what about Azure VPN access to avoid the problems? Maybe this isn't an option for you but I can imagine this could solve some of the problems you are facing in your scenario.

Debuggingcontext is something I will fix in the image one of the next days.
On the port number, the problem you have (I think) is that the PublicWebBaseUrl in the NAV configuration will need to know what URL points to NAV from the outside and if the port number is wrong, then it won't work. You might need to override some docker installation scripts to manage different foreign ports for the Web Client.
Security Certificate - haven't seen that one before.

Not sure what problem would be solved. If I want to to run multiple containers on Azure VM, I need to have multiple port mappings. Only one container can have port 443 mapped. If VS Code uses the port number that was specified in the server URL then there was one problem down.
The other one is the debugger context. That's a .Net problem and I guess it's due to the fact that NAV is running in an isolated process. So it would be nice to have a setting in VS Code to not debug (maybe there is already, or maybe I shouldn't use F5 to test the extension).
Correct me if I'm wrong. I'm on a steep learning curve these days...

What you are looking for here is really to be able to specify the ports like
-e webClientPort=61002
etc. for all 5 ports.
Might be a good idea.
/Freddy
From: Arend-Jan Kauffmann<EMAIL_ADDRESS>Sent: 15. august 2017 00:02
To: Microsoft/nav-docker<EMAIL_ADDRESS>Cc: Freddy Kristiansen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/nav-docker] Running on Azure VM works smoothly (#20)
Not sure what problem would be solved. If I want to to run multiple containers on Azure VM, I need to have multiple port mappings. Only one container can have port 443 mapped. If VS Code uses the port number that was specified in the server URL then there was one problem down.
The other one is the debugger context. That's a .Net problem and I guess it's due to the fact that NAV is running in an isolated process. So it would be nice to have a setting in VS Code to not debug (maybe there is already, or maybe I shouldn't use F5 to test the extension).
Correct me if I'm wrong. I'm on a steep learning curve these days...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fnav-docker%2Fissues%2F20%23issuecomment-322322100&data=02|01|freddy.kristiansen%40microsoft.com|910480495a7d42f901c008d4e3600f4c|72f988bf86f141af91ab2d7cd011db47|1|0|636383449086504375&sdata=ASbecxN03Qh%2BheUQ5%2B5kA3cwWbd4Q4yAdqoyDbzvaZQ%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKRqAxoPq7YAcJOV_hh6xbMtjcPPP3_nks5sYMPJgaJpZM4O2k0x&data=02|01|freddy.kristiansen%40microsoft.com|910480495a7d42f901c008d4e3600f4c|72f988bf86f141af91ab2d7cd011db47|1|0|636383449086504375&sdata=BFv2GgRcSwK6QvoREOhFjqOug8vmbYVsWXaSrisyh50%3D&reserved=0.

Interesting idea about port numbers, @freddydk.
Simple test with the docker container stopped proves that VS Code sends a request to the correct port number (from VS Code console). Obviously there is a redirect to the same url without port number. Could it have something to do with tenant redirection? (alternate tenant isn't used, but I don't know if the rewrite url config in IIS is enabled inside the container).
If port numbers need to be changed in the container, then we definitely need to know the mapping during start of the container. Would that be possible with a generated mapping (using the -P parameter)? If not, then we can only use predefined port mappings.
Regarding debugging: Ctrl+F5 is the solution, that is publish without debugging.

About the ports you are completely right, sorry for that. As you are running on W2k16 you cannot access natted resources from the host itself using the container names. This is actually possible on win10 where you can run (for example) 20 containers and access each of them using it's name like you are accessing separate VMs on the LAN: https://cont-01or https://cont-02 etc
But only from the host. So the VPN here could work (or I expect it could).

You cannot rely on the auto-mapping of random ports with -p – you would have to do that yourself in powershell when creating the container. NAV needs to know the public URLs.
For debugging – the missing config setting is
<NetFx40_LegacySecurityPolicy enabled="false"/>
In the server.exe.config file.
/Freddy
From: Arend-Jan Kauffmann<EMAIL_ADDRESS>Sent: 15. august 2017 00:11
To: Microsoft/nav-docker<EMAIL_ADDRESS>Cc: Freddy Kristiansen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/nav-docker] Running on Azure VM works smoothly (#20)
Interesting idea about port numbers, @freddydkhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffreddydk&data=02|01|freddy.kristiansen%40microsoft.com|4748faf1714d408f3d3908d4e3615f58|72f988bf86f141af91ab2d7cd011db47|1|0|636383454725389192&sdata=NypUDqtpWa93EbdXYRoVYeRX0EnMvtSsoXfzVGBju8M%3D&reserved=0.
Simple test with the docker container stopped proves that VS Code sends a request to the correct port number (from VS Code console). Obviously there is a redirect to the same url without port number. Could it have something to do with tenant redirection? (alternate tenant isn't used, but I don't know if the rewrite url config in IIS is enabled inside the container).
If port numbers need to be changed in the container, then we definitely need to know the mapping during start of the container. Would that be possible with a generated mapping (using the -P parameter)? If not, then we can only use predefined port mappings.
Regarding debugging: Ctrl+F5 is the solution, that is publish without debugging.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fnav-docker%2Fissues%2F20%23issuecomment-322324019&data=02|01|freddy.kristiansen%40microsoft.com|4748faf1714d408f3d3908d4e3615f58|72f988bf86f141af91ab2d7cd011db47|1|0|636383454725389192&sdata=IXJhhD8oHFEIDV2M9Cyc%2FAnM72alAaPgcDj8TlefukU%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKRqA-Lo555KAOBMkmm1bSLIUx_6bZ8hks5sYMX7gaJpZM4O2k0x&data=02|01|freddy.kristiansen%40microsoft.com|4748faf1714d408f3d3908d4e3615f58|72f988bf86f141af91ab2d7cd011db47|1|0|636383454725389192&sdata=fuMCHtZJW89OhObcRccjFXmhLII3HbMCp%2BE7dwb4P8g%3D&reserved=0.

I fully agree, an optional setting allow port definitions would be good. You could then expose them to the same ports on the azure host and map them to external azure ports. No confusion because of different port numbers internally and externally

The challenge will probably be how to expose those ports. You would only know during runtime which ports to expose, but the exposed ports have to be defined in the Dockerfile before docker build

On DebuggingContext - if you cannot wait - override SetupConfiguration.ps1 with this code:
$serviceTierFolder = (Get-Item "C:\Program Files\Microsoft Dynamics NAV*\Service").FullName
$serverConfigFile = Join-Path $ServiceTierFolder "Microsoft.Dynamics.Nav.Server.exe.config"
$serverConfig = [xml](Get-Content -Path $serverConfigFile)
$serverConfig.SelectSingleNode("//configuration/runtime/NetFx40_LegacySecurityPolicy").enabled = "false"
$serverConfig.Save($serverConfigFile)
and call the default SetupConfiguration.ps1 - then debugging in VS Code should work:-)
New images will be created one of the next days.

I can wait... have other stuff to do like work for customers...
Regarding exposing ports, I think the only option to run multiple docker containers is to use reverse proxy. Here is a good read how that could work, using Traefik proxy: https://stefanscherer.github.io/how-to-run-encrypted-windows-websites-with-docker-and-traefik/. With Docker Compose it should be easy to get a complete environment up and running.
Alternatively nginx could also do the trick. Seems that Traefik is the better suited and simpler to use.
Anyway, with a reverse proxy we would be able to run multiple containers on a VM and access them from outside based on the hostname, like container01.navdemo.net, container02.navdemo.net, etc. For a workshop setup this is an absolute must have.

Yes, traefic should solve the problem in a generic way. This would be probably the best way and it is true that in Docker worlds is being used very frequently.

Correct and as Tobias mentions, EXPOSE is a static thing, so the ports inside the container will always be the same. It is however a normal scenario (for purposes other than having multiple containers in one host) to change some ports, so for that I will add some parameters to specify to the NAV container which ports are used publicly to access WebClient, WinClient, Soap and OData. These are then used in the config file and the GETURL AL function will subsequently return correct values when you try to get the url for a page or like.
You still have the option to use traefik or other things.

I have been testing something similar (to be able to specify the ports) using the following in SetupVariables.ps1:
$portNavMgtSvc = if ([System.String]::IsNullOrEmpty($env:portNavMgtSvc)) { 7045 } else { $env:portNavMgtSvc }
$portNavSvc = if ([System.String]::IsNullOrEmpty($env:portNavSvc)) { 7046 } else { $env:portNavSvc }
$portNavSoapSvc = if ([System.String]::IsNullOrEmpty($env:portNavSoapSvc)) { 7047 } else { $env:portNavSoapSvc }
$portNavODataSvc = if ([System.String]::IsNullOrEmpty($env:portNavODataSvc)) { 7048 } else { $env:portNavODataSvc }
$portNavDevSvc = if ([System.String]::IsNullOrEmpty($env:portNavDevSvc)) { 7049 } else { $env:portNavDevSvc }

And then using the variables in the scripts instead of the fixed values.

the 7045 and 7049 ports shouldn't change (or rather - they are not referenced anywhere in a config file to be used from the outside). And the Web Client port should also be added (the default for that is depending on whether or not you use SSL though).
I will be checking in a fix today.

launch.json has a port setting which overrides the default port 7049. I was able to download symbols and publish an extension using a random port number, assigned by docker.

OK, I was thinking of a fully flexible solution so I have started changing all NAV ports. Web Client port should be added, of course, I haven`t included it yet.

Koubek - but you cannot EXPOSE them after build right?
AJ - korrekt 7049 is easy - you can connect to that on any port when mapped. The problem is that NAV needs to know the URL in the config file for connecting to NAV from the outside - NAV needs to know the ports for WinClient, OData, Soap and WebClient - else we will run into errors.

You can expose then with docker run --expose=[]

so
docker run -p 61000:7048 -e PublicODataClientPort 61000  ...
vs
docker run --expose [61000] -e Odataportsvc=61000 ...
I prefer no. 1 - so that inside the container ports are static and you can map them if you want - just remember to tell NAV what you map them to.

Option 1 would do the trick. Don't see a reason to change default port numbers inside the container.
Combined with the public hostname, it can create the correct base url's in the settings file (I assume that was what you were talking about).
Number of env variables are increasing rapidly. I recommend to start using the --env-file parameter. ;-)

I personally would prefer something like "Offset" or "Startport" so that we could e.g. do the following:
docker run -p 61000-61004:7045-7049 -e Startport=61000

and what about the Web Client?

61080 / 61443 maybe? So the full command would then be
docker run -p 61000-61004:7045-7049 -p 61080:80 -p 61443:443 -e Startport=61000

That becomes to much "magic" and something that is hard to understand if you didn't read this thread.
I will add the 4 optional parameters
publicWebClientPort
publicODataPort
publicWinClientPort
publicSoapPort
and if you don't specify them, then default values will be used.

wouldn't you need different ports for Web Client with and without SSL? And why didn't you include the DevServicePort and the MgmtServicePort?

No, the Web Client has one port.
Remember - the ONLY reason, that the NAV Container needs this info is, to write it in the customsettings.config in the settings:
PublicWebBaseUrl
PublicODataBaseUrl
PublicSoapBaseUrl
PublicWinBaseUrl
If it wasn't for these settings, the NAV server wouldn't care whether or not you mapped the ports.
Since Management and developer ports are not there - no need to specify values to the server it doesn't need

ok, understood. Thanks

Fixes checked in, generic image built and pushed.
Devpreview images for August and July being built and pushed now.
2017 and 2016 CU's will be built overnight.


Not Working with New Relic agent

[USER]:
Trying to set up Rollbar PHP client and New Relic agent.
New Relic agent stops tracking application performance, but works again when Rollbar PHP is removed.
internal: https://app.intercom.io/a/apps/rlisg82q/respond/inbox/488687/conversations/14332615800

I set up a project with New Relic and Rollbar but I wasn't able to reproduce the problem. I need more details on what exactly is happening.
I'm working on making the SDK more verbose right now here: https://github.com/rollbar/rollbar-php/issues/312
Once I add more debugging info, let's try to find out what exactly is happening.

I attacked this today too from the New Relic side, and I haven't been able to reproduce the issue either. I'll let you know if I find anything!

Hi! I was the one that reported it some days ago. If helps I can attach our new-relic / php config.

@theUniC that would be awesome. Also, I just merged into master  a new configuration option that will let you get more details on what's happening inside the SDK. So,

Switch to using rollbar/rollbar-php:master instead of the last tag v1.3.6.
Configure your Rollbar with

'verbosity' => \Psr\Log\LogLevel::DEBUG

Post the contents of your /tmp/rollbar.debug.log.

You can find more about the verbosity config option here: https://github.com/rollbar/rollbar-php

@theUniC Any more details on this?

@ArturMoczulski It's a New Relic issue. We investigated it alongside them and found out that by completely disabling error tracing on the newrelic-agent things were working out again. I suspect they are doing things internally when register_shutdown_function callbacks are invoked, but don't have any more information.

I see. I'm going to close the issue here now.
However, you probably still need to fix this situation for yourself - I would love to hear what you guys come up with to have the New Relic agent work with Rollbar PHP SDK.


Fix typo in README.md

Hi @Sintendo
thanks for your contribution. Do you plan to sign the Oracle Contributor Agreement? If so, I'll wait for that and merge your PR afterwards.
If not, I can fix the type as part of one of my PRs under my name.
Unfortunately, we have to follow this legal process even for small fixes like this one.
Best,
Christian

Wasn't aware of the OCA. Feel free to fix this in a PR of your own.

Hi @Sintendo
I have a PR ready in our internal CI. Should be merged later today. I'm closing this PR here.
Thanks again for your contribution. Feel free to either sign the OCA for direct contribution, or report "issues" so we can fix them.
Thanks,
Christian


Crash when quitting app on macOS

There seems to be some crash when the app is quit.
I get this every time I quit the app.

Just close the app window and quit the application.
Actual behavior

To reproduce
Quit the app

Happens every time I quit the app.
System information
macOS 12 on Apple Silicon.

Dunno

Addendum: When you quite the QT app, you get a dialogue asking to not shut down the computer until the app quits. Would be good to have that if applicable.
Might or might not be related.
Can provide screenshot if required.

Mind checking if it happens with a self compiled binary?

attaching these lines from debug.log
2023-04-17T12:55:40Z GUI: qrc:/qml/components/ConnectionSettings.qml:56: TypeError: Cannot read property 'server' of null 2023-04-17T12:55:40Z GUI: qrc:/qml/components/ConnectionSettings.qml:43: TypeError: Cannot read property 'natpmp' of null 2023-04-17T12:55:40Z GUI: qrc:/qml/components/ConnectionSettings.qml:30: TypeError: Cannot read property 'upnp' of null 2023-04-17T12:55:40Z GUI: qrc:/qml/components/ConnectionSettings.qml:17: TypeError: Cannot read property 'listen' of null

cc @jarolrod @john-moffett

Don't get this crash on the build from #306


Add school file

Kirikkale University

@mehmetenesturan I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Do you have any plan to update slot with v-slot directive?

Popper did not work when I tried to replace slot with v-slot. slot is deprecated in Vue 2.6.0, Do you have any plan to use new syntax (v-slot)?

I looked into this a bit, I think it's related to vuejs/vue#5427. This line no longer works

#https://github.com/vuejs/vue/issues/9421

Still doesn't seem to work.
When I do:
<template v-slot:reference>
    <div class="popover">
        Hover me
    </div>
</template>

It doesn't work.

Still doesn't work with new slot syntax ...

In 2.6.0, we introduced a new unified syntax (the v-slot directive) for named and scoped slots. It replaces the slot and slot-scope attributes, which are now deprecated, but have not been removed and are still documented here. The rationale for introducing the new syntax is described in this RFC.

    <Popper toggle="":options="{ placement: 'top' }">
      <BaseButton @click="toggleMenu" slot="reference">
        <template v-slot:icon>
          <ChecIcon :icon="isOpen ? 'up' : 'down'" />
        </template>
      </BaseButton>
      <BasePopover
        @option-selected="handleSelectOption"
      >
        <!--
          @slot Provide BaseOption instances here
        -->
        <slot />
      </BasePopover>
    </Popper>

I believe this is due to the fact that elm is undefined on Vnodes with new slot syntax. See this closed issue and the links to comments by Evan etc. https://github.com/vuejs/vue/issues/10450#issuecomment-526106397. Vue-popper won't work with Vue 3 unless you're passing the reference as a prop instead of using the slot.

I've rewritten the component completly in vue 3.0 composition api style, it works with the new scoped slots (v-slot) directive.  If you want a copy, let me know.

I've rewritten the component completly in vue 3.0 composition api style, it works with the new scoped slots (v-slot) directive.  If you want a copy, let me know.

I've rewritten the component completly in vue 3.0 composition api style, it works with the new scoped slots (v-slot) directive.  If you want a copy, let me know.

I'd love to see this and have it available via npm to use.


Chrome OS client for the chromebooks that do not get Android support

Would it be possible to provide a Chrome OS client for the chromebooks that do not get Android support?
You claim to support Chrome OS 65+ .. but for example the Acer C720p does run 65+, but does not support Android apps so it can not use your Android App to provide VPN

I have a Chromebook with the same problem, so I can sympathize, but unfortunately this is very difficult. Also, all new Chromebooks come with Android app support, so this work would only be for the shrinking fraction of Chromebooks that don't support Android apps.


Questions regarding migrating from 3.3.3 to 4.0.1

Hi, I'm struggling migrating from 3.3.3 to 4.0.1
I have an application with an encrypted database (v.3.3.3)
   if(some logic to make sure it's only called once ){
      await  this.sqlite.setEncryptionSecret({ passphrase: "RANDOM" });
   }
   this.sqlite.createConnection(dbName, true, "secret", 1);

which is working fine and the db is encrypted successfully
here are my steps to migrate to (4.0.1)
1- I added in plugins in capacitor.config.json
 "CapacitorSQLite": {
          "iosDatabaseLocation": "Library/CapacitorDatabase",
            "iosIsEncryption": true,
            "iosKeychainPrefix": "MY_APP_NAME"
        }

2- I'm calling isSecretStored in my code
    const isStored =(await this.sqlite.isSecretStored())?.result;

   if(!isStored ){
    await  this.sqlite.setEncryptionSecret("RANDOM");
 }

 this.sqlite.createConnection(dbName, true, "secret", 1, false); // added the fifth parameter

Here is what I understand about what is happening with these settings
1- the old database will be moved from Documents to Library/CapacitorDatabase automatically (which is correct and tested)
2- iosIsEncryption is used when our old database is encrypted
3- isSecretStored is used to migrate the old keychain value to the new one with the prefix MY_APP_NAME
I get an error openOrCreateDatabase: Cannot open DB
I think it's because the old secret key didn't migrate to the new key with the prefix, any advice ?

@scr2em have you replace MY_APP_NAME by the name of your app as defined in your package.json.  otherwise it is difficult to understand not having the code. May be should have been better to migrate to 3.7.0 and when everything is done and checked to migrate to 4.1.0

@jepiqueau Thank you for your fast reply, MY_APP_NAME is the same as appName in capacitor.config.json which is very different than name in package.json. is it required that it iosKeychainPrefix should be the same as name in package.json ?

@scr2em no but it should be unique so that why i though that the app name what the right thing to take

@jepiqueau
ok so about these
1- the old database will be moved from Documents to Library/CapacitorDatabase automatically (which is correct and tested)
2- iosIsEncryption is used when our old database is encrypted
3- isSecretStored is used to migrate the old keychain value to the new one with the prefix MY_APP_NAME
do I understand them correctly?
also how can I test that the keychain value has been migrated to the new one with the prefix ?

@scr2em  You understand correctly. there is no way to test the keychain value except in debugging your app by simulating a changeSecureSecret an put a breakpoint in line 28 of the class utilsSecret if it is going to line 29 means that the prefix is in and after this you stop the app.

@scr2em any progress on this? can i close the issue

@jepiqueau I'm still stuck on it. but sure you can close it
also can you open Discussion on this repo ?
https://docs.github.com/en/discussions/quickstart

@scr2em i did open the Discussion on this repo as you suggested. i am not happy of closing the issue if you are still stuck on it. May be you could share your app?

@jepiqueau I'm still debugging till I reach the root cause of this issue. if I need something I'll start a discussion.
also when I get some free time I'll help improving the current docs.
Thanks

@scr2em OK thanks keep me aware if you find something which can be useful for others.


Error in render: "TypeError: Cannot read property 'filter' of undefined"

I have just updated the latest package and it gives me error Error in render: "TypeError: Cannot read property 'filter' of undefined" and Cannot read property 'querySelector' of undefined

@hmuhdkamran please get me more details:

how to use vuesax on your project?
the error is on what component?
please print the console.log output
what version have in your package.json

Thanks!

Sorry the error is in vs-table,
I am using the vs-table like
<vs-table :data="data">       <template slot="thead">         <vs-th> Country </vs-th>         <vs-th> Currancy </vs-th>         <vs-th> Operation </vs-th>       </template>       <template slot-scope="{data}">         <vs-tr :key="indextr" v-for="(item, index) in data" >           <vs-td :data="item.country"> {{item.country}} </vs-td>           <vs-td :data="item.cur"> {{dataitem.cur}} </vs-td>           <vs-td :data="item.id">             <button @click="edit(item)">Edit</button>             <button @click="delete(item)">Delete</button>           </vs-td>         </vs-tr>       </template>     </vs-table>
currently I am using v3.4.6

@hmuhdkamran i try to simulate your scenario, look:
https://codesandbox.io/s/qzkj15l52w
one change i made, you set <vs-tr :key="indextr" v-for="(item, index) in data" > but indextr not exists, make sure to use index instead at indextr

Thanks That did the trick, We are planning to use Vuesex in your project,
but it still in infant state, plus some essential controls are still
missing, one of them is Date and time.
I constantly looking at this framework when It come with DateTime controls
I will definitely try it in my application.
On Sat, Oct 6, 2018 at 5:38 PM Bruno Henry<EMAIL_ADDRESS>wrote:

@hmuhdkamran https://github.com/hmuhdkamran i try to simulate your
scenario, look:
https://codesandbox.io/s/qzkj15l52w
one change i made, you set  but indextr not exists, make sure to use index instead at indextr
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lusaxweb/vuesax/issues/261#issuecomment-427570455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbBj3qLLhOnQH8v-Y_WTVe8LEr_5276Mks5uiKQ-gaJpZM4W8OQb
.

--
COMBINE CHIPSoft Corporation
(0333/0313)-9112845
http://combinesoft.net/

Thank you very much for using vuesax in quick updates we are going to add the timepicker and datepicker components


Anchor tag with button style causes text to be to high

The images below show it all.

Greetings @sean-hill!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


Multiple root folders

Hello! Thank you for the great tool. Is there built in way to scan several "root" folders for the same solution. We have solution that consists of several libraries that designed to be executed as "plugins" by external host app, so there is no single composition root for that project. Another use case - are unit test projects. Of course, we are not going to ship their code but it is good to know if we have issues there.
I guess I can use the following https://github.com/microsoft/infersharp/blob/main/RUNNING_INFERSHARP_ON_WINDOWS.md to achieve my goal, but this requires some tricky path translations in case if I want it to run on any user machine.

Hi, I think the quickest mitigation is to modify the execution script yourself. Basically the script copies all the .dll and .pdb files from an input path into a staging folder like this. You can modify it to allow multiple input paths and iterate through them with the same logic.

I thought it sits inside WSL container and will be rewritten with next release?

Yes, this is just a mitigation for you before our next release. You can modify the script in your WSL instance in the meantime. I will mark this as a feature request.

Thank you


update to etcd 2.2.1

Hi,
If you are interested, I've updated to the latest etcd google version and it works like a charm ;-).
Rgds,

Thanks for that :)

You're welcome :)


Add a Packit configuration for Fedora packaging

all pull requests are build-tested in https://copr.fedorainfracloud.org
new releases will create pull requests in https://src.fedoraproject.org/rpms/httpie

As discussed with @BoboTiG over email. Not tested yet.
The CI build will fail because of https://github.com/httpie/httpie/pull/1057#issuecomment-857684958 -- once that is fixed, I'll rebase this.
The service needs to be enabled first: https://packit.dev/docs/packit-as-a-service/

If I understand well, there is no more need for us to provide the httpie.spec file because the one from https://src.fedoraproject.org/rpms/httpie/raw/rawhide/f/httpie.spec will be used, right?

Indeed. As a downstream maintainer, I'd like to avoid keeping the specfile in upstream, as it makes my life harder than necessary.

And it makes mine easier, so it is perfect :)

Packit GH app is installed ;)

Namespace is not allowed!

Let me see... :/

https://github.com/packit/packit-service/issues/997 seems to indicate that "Namespace is not allowed!" means that some real human from the packit team will get notified that we attempted to use it here and they need to allow github.com/httpie manually to verify we are not scumbags. Is that correct @TomasTomecek?

Codecov Report

Merging #1086 (1a727fb) into master (9792513) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 1a727fb differs from pull request most recent head aa265d2. Consider uploading reports for the commit aa265d2 to get more accurate results

@@           Coverage Diff           @@
##           master    #1086   +/-   ##
=======================================
  Coverage   95.84%   95.84%
=======================================
  Files          61       61
  Lines        4020     4020
=======================================
  Hits         3853     3853
  Misses        167      167

Continue to review full report at Codecov.

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

/packit build

packit/packit-service#997 seems to indicate that "Namespace is not allowed!" means that some real human from the packit team will get notified that we attempted to use it here and they need to allow github.com/httpie manually to verify we are not scumbags.

Fixed in https://github.com/packit/notifications/issues/361

It appears to work. The build failures are real and they are related to https://github.com/httpie/httpie/pull/1057#issuecomment-857684958

Thanks a lot @hroncok !

Awesome stuff 📦🎉🚀

packit/packit-service#997 seems to indicate that "Namespace is not allowed!" means that some real human from the packit team will get notified that we attempted to use it here and they need to allow github.com/httpie manually to verify we are not scumbags. Is that correct @TomasTomecek?

you are correct, we had to put this namespace on a allowlist :)
https://packit.dev/docs/faq/#can-i-use-the-packit-service-as-soon-as-i-install-it-into-my-repository
sorry for a late reply - summer is coming :) finally tracking down all the notifications
@hroncok thank you for setting this up!!


feat(tag-group): create new component

Related Ticket(s)
#5996
Description
Create new TagGroup Component in both Web Components and React Wrapper. This also includes Sandbox examples.

Changelog
New

TagGroup component for WC and React Wrapper

Web components look good! React wrapper showing a bit of an issue when the dropdown selects Carbon tag:


Breadcrumbs show wrong CURIE capitalisation

is FBBT:123, should be FBbt:123
Example: https://www.ebi.ac.uk/ols4/ontologies/fbbt/classes/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FFBbt_10000000?lang=en

(copy paste works correctly)


Create tests for the automated chatflows

Tests should cover all solutions.

marketplace

[x] wiki
[x] website
[x] blog
[x] mattermost
[x] discourse
[x] taiga
[x] peertube
[x] gitea
[x] cryptpad

solutions

[x] network
[x] pools
[x] ubuntu
[x] kubernetes
[x] minio
[x] expose solutions
[x] generic flist
[x] monitoring
[ ] domain delegation
[ ] 4 to 6 gateway

3bot deployer

[x] create
[ ] extend
[x] recover

[ ] Tests should be run nightly on github actions.
[x] All tests should take integration marker.

i can't create ns record in the test

extend threebot work fine in frontend but get this error when try by script


import error no module named piston

I'm attempting to install piston on a mini install of ubuntu.
I've downloaded the dependencies necessary to do the install.
And according to the installation piston installed.
Now when I run my script on the command line I get an error.
python3 bot.py
traceback (most recent call last):
File "bot.py" , line 8, in
from piston.steem import Steem
ImportError: No module named 'piston'
I can only assume that something didn't install correctly in the right place.
Any ideas?

well.....
Now I tried steem.steem
and the bot runs. however i have to comment out Broadcasting error.
cannot import name broadcasting error

Can you give us a full code snippet?

Hi xeroc.
I'm attempting to get fyrstikkens oprah bot to run.
https://steemit.com/upvote-bot/@fyrstikken/updated-winfrey-upvote-bot-will-vote-for-everyone-who-post-on-steemit-with-a-2-upvote-except-posts-flagged-by-cheetah-or
`# This Piston/Python-Bot will vote for everyone who post except those flagged by
@cheetah or @steemcleaners. By Default the Upvote is set at 2%
Created by: @furion, @contentjunkie, @anyx, @inertia, @fyrstikken
Make sure you have the latest version of Python and Piston installed
See Original Post for details and step by step process.
For help come on http://steemspeak.com or http://steemit.chat
from piston.steem import Steem
from piston.steem import BroadcastingError
import threading
import time
import random
import csv
my favorite blogs on steemit
top_writers = []
add my favorites
my_favorites = []
Skiplist functionality has not been added yet, this will be your personal blacklist
with open('skiplist.txt', mode='r') as infile:
reader = csv.reader(infile)
for rows in reader:
v = rows[0]
top_writers.append(v)
Add your Account, Posting key for voting and/or Active Key for tipping
my_subscriptions = top_writers + my_favorites
account = ["your-account-name"]
posting_key = ["your-private-posting-key"]
active_key = []
This is the range (in seconds) in which your bot will cast a vote
vote_delay = random.randrange(1200,1800)
upvote_history = []
def feed():
Starting The Bot
print("Upvote Bot Started - Waiting for New Posts to Upvote!")
steem = Steem(wif=posting_key[0])
for comment in steem.stream_comments():

    if True:

Just making sure we vote for the post and not a comment.
        if comment.depth == 0:

check if we already upvoted this. Sometimes the feed will give duplicates.
            if comment.identifier in upvote_history:
                continue

            print("New post by @%s %s" % (comment.author, url_builder(comment)))
            workerThread = threading.Thread(name=comment.identifier, target=worker, args=(comment,))
            workerThread.start()

Tipping Function send $0.001 in STEEM
def send_a_tip(author):
steem = Steem(wif=active_key)
steem.transfer(author, 0.001, "STEEM", memo="Keep Blogging", account=account)
def url_builder(comment):
return "https://steemit.com/%s/%s" % (comment.category, comment.identifier)
def worker(worker_comment):
time.sleep(vote_delay)
try:
for (k,v) in enumerate(account):
worker_steem = Steem(wif=posting_key[k])
upvote_comment = worker_steem.get_content(worker_comment.identifier)
Checking if Post is flagged for Plagarism & Spam
     names = ['steemcleaners', 'cheetah']
     for avote in upvote_comment['active_votes']:
       if (avote['voter'] in names and avote['percent'] < 0):
        print("====> Upvote Skipped - Post Flagged by Cheetah or Steemcleaners")
        return False

If the post has not been flagged by Cheetah or Steemcleaners, it will now upvote the post
with a 2 % upvote
     upvote_comment.vote(2, v)

Upvote has now been done and it will now print a message to your screen:
     print("====> Post Upvoted Successfully!")
     upvote_history.append(upvote_comment.identifier)
 except BroadcastingError as e:
   print("Upvoting failed...")
   print("We have probably already upvoted this post before the author edited it.")
   print(str(e))

   if upvote_comment.author in my_favorites:
     send_a_tip(upvote_comment.author)
     print("====> Sent $0.001 STEEM to @%s" % upvote_comment.author)

if name == "main":
while True:
try:
feed()
except (KeyboardInterrupt, SystemExit):
print("Quitting...")
break
except Exception as e:
traceback.print_exc()
print("### Exception Occurred: Restarting...")
If you want to add more complexity to the Bot, feel free to do so and share it with
The Steemit community on your own Blog.
`
I think I've narrowed it down to it being that this script will not run with the latest version of piston.
Any suggestions?

Solved the issue by completely uninstalling steem
and using pip3 install steem==0.3.1
Seems like the latest version of steem has broken almost everyone's scripts.

Hello,
I have the same error.
When I try pip3 install steem==0.3.1
Collecting steem==0.3.1   Could not find a version that satisfies the requirement steem==0.3.1 (from versions: 0.17.0, 0.17.1) No matching distribution found for steem==0.3.1

Please see this announcement:
https://steemit.com/chainsquad/@chainsquad/python-steem-rebranding-to-piston-libpiston-cli