text:
File upload is broken when SSL is offloaded on a reverse proxy

Used configuration
CommunityServer is being launched from docker container like this:
onlyoffice-cs:
  image: onlyoffice/communityserver:9.5.2.546
  restart: always
  stdin_open: true
  tty: true
  container_name: onlyoffice-cs
  external_links:
   - onlyoffice-ds:document_server
  ports:
   - "172.17.0.1:5080:80"
   - "172.17.0.1:50443:443"
   - "5222:5222"
  volumes:
   - /home/████████/server_data/onlyoffice/cs/logs:/var/log/onlyoffice
   - /home/████████/server_data/onlyoffice/cs/data:/var/www/onlyoffice/Data
   - /home/████████/server_data/onlyoffice/cs/mysql:/var/lib/mysql
   - /home/████████/server_data/onlyoffice/ds/data:/var/www/onlyoffice/DocumentServerData
  environment:
   - MYSQL_SERVER_HOST=172.17.0.1
   - MYSQL_SERVER_DB_NAME=onlyoffice_cs
   - MYSQL_SERVER_USER=root
   - MYSQL_SERVER_PASS=████████
   - MYSQL_SERVER_PORT=3306
   - DOCUMENT_SERVER_PORT_80_TCP_ADDR=oods.████████

It is accessed through nginx reverse proxy which offloads SSL and then redirects plain http to 172.17.0.1:5080 like this:
map $http_upgrade $connection_upgrade {
    default upgrade;
    ''      close;
}

server {
    listen 80;
    server_name office.████████;
    return 301 https://$server_name$request_uri;
}

server {
    listen 443 ssl;
    ssl on;
    server_name office.████████;
    ssl_certificate ████████;
    ssl_certificate_key ████████;

    location = /favicon.ico {
        return 204;
        access_log off;
        log_not_found off;
    }

    location / {
        client_max_body_size    4G;

	proxy_pass http://172.17.0.1:5080;
	proxy_read_timeout 1800;
	proxy_connect_timeout 1800;
	proxy_set_header Host $http_host;
	proxy_set_header Upgrade $http_upgrade;
	proxy_set_header Connection $connection_upgrade;
	proxy_set_header X-Forwarded-Proto $scheme;
	proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
	proxy_set_header X-Real-IP $remote_addr;
    }
}

Steps to reproduce
In Documents click upload button and select any file.
Observed behavior
I'm getting error like this:
Mixed Content: The page at 'https://office.████████/products/files/#1' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://office.████████/products/files/ChunkedUploader.ashx?uid=73b614d569e448d4ac753a7f19507a62'. This request has been blocked; the content must be served over HTTPS.

I've tinkered a little with client JS code and found out that this http link to ChunkedUploader is being sent by server in a response to /api/2.0/files/1/upload/create_session.json call. As JS is minified on the client I can't provide exact file/line of code where it happens. Code in the debugger looks like this:
 var i = jq.format("{0}files/{1}/upload/create_session.json", ASC.Resources.Master.ApiPath, n.fid)
          , t = null;
        return jq.ajax({

The whole response that I'm getting looks like this:
{
  "success": true,
  "data": {
    "id": "b7d920e3776742d9832367fcbcd37131",
    "path": [
      1
    ],
    "created": "2017-12-17T14:46:05.832996Z",
    "expired": "2017-12-18T02:46:05.832996Z",
    "location": "http://office.████████/products/files/ChunkedUploader.ashx?uid=b7d920e3776742d9832367fcbcd37131",
    "bytes_uploaded": 0,
    "bytes_total": 12354
  }
}

Expected behavior
File upload working as intended :grin:
Side notes

If I place breakpoint in the aforementioned code and correct returned link by hand the rest part of file upload is working as intended.
As seen from my nginx virtualhost config I pass X-Forwarded-Proto header, but seems that it's not being considered by the server when generating ChunkedUploader link

Closed for a reason - there has been no activity for a long time.

authors: Carazyda kykc

repo: ONLYOFFICE/CommunityServer


text:
Fix spacing

Describe the bug
Refine and document spacing

Sort out default margins
More spacing above headings

Additional context
Links to the need for a spacing scale

Part of spacing work - not currently a priority

This ticket is duplicated in #1994 and #1935  - will close this ticket as work is already covered in other tickets and awaiting UCD.

source: gharchive/issue

authors: daniellecorke yatesn

repo: ONSdigital/design-system

license: MIT

license_type: permissive

license_source: github-api


text:
Add current date default for started_at

Relates to: https://github.com/ONSdigital/eq-survey-runner/pull/1732
Ensures that the runner doesn't error when it receives a non-date string for 'started_at'
I used the current date as the default, since it doesn't seem to like empty strings. This is probably better anyway.
I also ran gofmt.

Discussed with Adam, decided to remove since we shouldn't have started_at in metadata anymore

source: gharchive/pull-request

repo: ONSdigital/go-launch-a-survey


text:
Bump common entities version

Motivation and Context
If two processes attempt to simultaneously update the data on an @Entity then the whole data row will be replaced by whatever is in memory, by default. By using the @DynamicUpdate Spring Data JPA annotation, we indicate to JPA, and in turn to Hibernate, that we want an update SQL statement to be created, which will only update the values which have changed by using a 'proxy' object... whenever we call a setter on the proxy, JPA/Hibernate knows that we wish for that data item to be updated, but none of the other items.
By using @DynamicUpdate we suffer a minor performance penalty, because the SQL statement will be prepared dynamically rather than pre-prepared and cached... but the benefit is clear in terms of not over-writing changes which happened simultanously.
What has changed
Added @DynamicUpdate to all the JPA entities.
How to test?
Should be zero noticeable change, but you could try doing simultaneous refusal and invalid on cases, and check that those flags are not 'lost'... in Census we did see (in very rare circumstances) that we lost the flags because of this problem.
Pretty much just run the ATs.
Links
Trello: https://trello.com/c/4TzJNke2

 enable dynamic case updates (3)

repo: ONSdigital/ssdc-rm-caseprocessor


text:
test PR with Github API

Please pull this in!

Docs Build status updates of commit 996a51d:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 975417a:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
DocsArchive-15.0.0.0Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_c34d7e80-4749-4daa-969f-ab0cd04a02bc/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_c34d7e80-4749-4daa-969f-ab0cd04a02bc.git

Docs Build status updates of commit 6053191:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


text:
DocsArchive-1.0.0.0Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/105 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 62f5ace:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-29-04-01-25 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-29-04-01-25

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

authors: VSC-Service-Account v-alji


text:
Remove c++ api on read the docs

Basically cherry-picking changes in #458 into the dtk-3.0 branch

Codecov Report

:exclamation: No coverage uploaded for pull request base (dtk-3.0@fa22cbe). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             dtk-3.0     #460   +/-   ##
==========================================
  Coverage           ?   96.83%
==========================================
  Files              ?      123
  Lines              ?     9093
  Branches           ?        0
==========================================
  Hits               ?     8805
  Misses             ?      288
  Partials           ?        0

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 fa22cbe...e355b66. Read the comment docs.

repo: ORNL-CEES/DataTransferKit

license_source: bigquery


text:
Fix warnings

Fix a couple of warnings:

shadowing variables
suggest final

Codecov Report

Merging #160 into master will not change coverage.
The diff coverage is 88.88%.

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   89.09%   89.09%
=======================================
  Files          53       53
  Lines        2503     2503
=======================================
  Hits         2230     2230
  Misses        273      273

Impacted Files
Coverage Δ

include/mfmg/dealii/amge_host.templates.hpp
92.3% <100%> (ø)
:arrow_up:

include/mfmg/dealii/dealii_solver.hpp
100% <100%> (ø)
:arrow_up:

...nclude/mfmg/dealii/dealii_matrix_free_smoother.hpp
100% <100%> (ø)
:arrow_up:

...de/mfmg/dealii/dealii_trilinos_matrix_operator.hpp
100% <100%> (ø)
:arrow_up:

tests/test_hierarchy_helpers.hpp
65.33% <71.42%> (ø)
:arrow_up:

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 69cdc4a...e4665e9. Read the comment docs.

authors: Rombur codecov-io

repo: ORNL-CEES/mfmg


text:
An error occurred during the code running for the latest Mtex (V2.3.0)

Dear developers,
Thank you very much for providing the powerful tool "ORTools" for EBSD analysis. There is a problem when I run the code of  "ORTools_Example10.m".
Description of the bug
When I run " [newGrains,~] = computeVariantGrains(job);" , an error occurred .
The error is "Error using cat
The dimensions of the arrays to be concatenated do not match.
Error in dynProp/cat (line 35)
dp.prop.(fn{i}) = cat(dim,s.(fn{i}));
Error in EBSD/cat (line 16)
ebsd = cat@dynProp(1,varargin{:});
Error in dynProp/vertcat (line 48)
dp = cat(1,varargin{:});
Error in computeVariantGrains (line 67)
newEBSD = [cEBSD; remainingEBSD];
Error in ORTools_example10 (line 125)
[newGrains,~] = computeVariantGrains(job);"
ebsd data
I have tried different data but the error is still existing.
MTEX version:

Version [2.3.0]

I am looking forward to your reply.

Hi @FeC3-pearlite,
Thank you for getting in touch and letting us know of a possible bug.
To reproduce your error and diagnose the fault, I'd be very grateful if you could please upload your script and the EBSD map being called for parent grain reconstruction by your script.
Warm regards,
Azdi

Dear developers,
It's a surprise to receive the reply from you so quickly, thank you!
There is a mistake in my statement last time. The Mtex version I have used is V5.10.0.
I just used the original script named 'ORTools - Example 10' in the latest version of ORTools. I have upload the ebsd map and the script in the attachments. There is similarly a mistake in the function 'computeVariantGrains'. I also tried V2.2.0 ORTools based on Mtex 5.10.0, but the error doesn't occur.
EBSD map

Script:
https://github.com/FeC3-pearlite/ORTools-example-10/blob/main/ORTools_example10.m
I am looking forward to your reply.
Yours sincerely,
Jack Liu


text:
Feature comparision matrix

Feature
KeyCloak
Authentik
Zitadel
Authelia
Kanidm

Container ready
✅
✅
✅
✅
✅

SAML2 ID integration
✅
✅
✅
❌
❌

Primary codebase
Java❓
Python ✅
Go ✅
Go ✅
Rust ✅

Total Commits
>23k
>13k
~3k
~5k
~1.7k

Standardized backing services

✅
❌

Feature 5
...

After evaulation, Authentik was chosen for the PoC


text:
Osvr unity version check

OSVR Unity editor window now checks if a new OSVR-Unity build is available and if so, adds a button link to the downloads page.

CI updated, tested that this works with or without internet connection.

repo: OSVR/OSVR-Unity


text:
Where is the content of this guide

As discussed at the OWASP Projects Mini-Summit in Cambridge?
Wasn't the idea to put the content here?

hello? any developments?

I'm interested in contributing to V2.0. Are there plans on adding it to this repo?

not sure,  @addy-s what is the status of this project?
should we remove the repo?

@DinisCruz If there interest from other people, I don't see why we should remove.
Sadly, I had started a new job and a new study program shorty after hour discussion two years ago, so it's unlikely that I'll be able to do anything on this one.

@addy-sh ok, so where are the original files? (i.e the files with the content you created)

authors: DinisCruz addy-sh clayball

repo: OWASP/CodeReviewGuide


text:
Integrate new Armada Protocol SDK backend with the FE to get user positions and position details

Description
Integrate new Armada Protocol SDK backend with the FE to get user positions and position details
Changes

Integrate new Armada Protocol to get user positions and position details

Benefits

[List the benefits of these changes]

Testing
[Describe how these changes were tested or how they can be tested]
Next steps

[List any follow-up actions or considerations]

Additional Notes
[Any additional information, context, or notes for reviewers]
Please review and provide any feedback or suggestions for improvement.

Summary by CodeRabbit

New Features

Enhanced wallet connection and message signing capabilities in the AccountKit component.
Introduced new dependencies to improve GraphQL operations and project compatibility.

Bug Fixes

Improved consistency in TypeScript type definitions and interfaces for better readability.

Chores

Updated project dependencies to streamline development and enhance functionality.

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

Project coverage is 70.37%. Comparing base (19463b1) to head (2f11be0).

:exclamation:  There is a different number of reports uploaded between BASE (19463b1) and HEAD (2f11be0). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (19463b1)
HEAD (2f11be0)

4
1

Additional details and impacted files
@@             Coverage Diff             @@
##              dev     #491       +/-   ##
===========================================
- Coverage   83.68%   70.37%   -13.31%
===========================================
  Files         288      129      -159
  Lines        2923     1357     -1566
  Branches      175       78       -97
===========================================
- Hits         2446      955     -1491
+ Misses        467      402       -65
+ Partials       10        0       -10

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

authors: codecov-commenter piotrwitek

repo: OasisDEX/summerfi-monorepo


text:
Trainers Way of Life not working

Professor Ras keeps saying to read the rules

You need to have the pokes she asks you to go get in your team. Otherwise how can she inspect them?

repo: ObliqueNET/Terra


text:
NestedField and SummedField

Hello,
I'm trying to use at the same time 2 velocity sources (currents and Stokes drift) and grid nesting. I have 3 nested Nemo grids that I can use fine with Parcel 2.2 (following the Nemo and Nesting tutorials). I also can combine currents with Stokes drift obtained from CMEMS (so yet another grid), when I use just 1 of the 3 Nemo grids, e.g. following the SummedField tutorial, or alternatively using 2 kernels (as in the North Sea example).
The difficulties start when I want to use together grid nesting and currents+stokes drift. I tried the following:

use summed fields and nestedfields in order to use just 1 kernel (advectionRK4). 2 variants are possible,
** a Nestedfield of 3 summedfields, each one is the sum of the respective Nemo velocity and (always the same) Stokes drift velocity
** a summedfield of a NestedField and the stokes drift. Neither solution seems to work. I suppose NestedField wants Fields, not SummedFields ; and the same the other way around.
create a fieldset containing NestedField for U and V, then adding Stokes drift as a separate VectorField in the fieldset; Then I would use 2 kernels, advectionRK4  + StokesDrag (as in the North Sea examples).
** I cannot add the VectorField (Ustokes, Vstokes) directly to the fieldset containing the Nemo nestedfield, because it expects to find the fieldset.U.grid attribute, which does not exist (and indeed U is a NestedField instead of a regular Field)
** I added the Stokes drift Vectorfield to the fieldset1, the fieldset corresponding to one of the 3 Nemo fieldsets, then wrote something like
fieldset.UVs = fieldset1.UVs    (where UVs is the VectorField containing the Stokes drift)
Here, an error is raised when I try to add the StokesDrag kernel to the kernels variable, because the Field does not have the ccode_name attribute.
** If I first add both components of the Stokes drift velocity fieldset_total.Us = fieldset1.Us (and the same for V), then it accepts adding the StokesDrag kernel, but fails at the pset.execute line with the error that the RectilinearZGrid object does not have the update_status attribute.

Is there a preferred way to use both nesting and 2 velocity sources ? Or anything else that I forgot or did wrong ?
My source code is in the attached zip file ; the datafiles are large (~1GB) but I can share them somewhere on the web if it helps.
Thank you !
tracking.zip

Thanks @lucvandenbulcke, for reporting this bug. I think you're the first user who wants to use both NestedFields and SummedFields at the same time... We just haven't developed/tested this feature yet
I can have a quick look if it's as easy as making sure that NestedFields also accepts SummedFields.
If that turns out to be difficult to implement, then perhaps a solution is to put the SummedFields part explicitly in your kernels. So instead of a line like
(u1, v1) = fieldset.UV[time, particle.depth, particle.lat, particle.lon]

use something like
(u11, v11) = fieldset.UV[time, particle.depth, particle.lat, particle.lon]
(u12, v12) = fieldset.UVs[time, particle.depth, particle.lat, particle.lon]
u1 = u11 + u12
v1 = v11 + v12

which is what Parcels does under the hood anyways, for SummedFields. Note that this is different from defining a separate StokesDrag_RK4 kernel and then appending to the normal AdvectionRK4 as you did in your code, as in that case the particle will first be updated with UV and only after its location is updated will it 'feel' Stokes. That can lead to a different trajectory than computing UV and UVs at the same times

Thanks @erikvansebille for your answer.
I tried to apply your method. I create fieldset1, fieldset2, fieldset3 (each from Nemo), and then fieldset using NestedFields
Then I try to read in the Stokes drift. I cannot save the UVs VectorField to fieldset, because the "NestedField" has no attribute 'grid'
I then save UVs in fieldset1 , and just write fieldset.UVs=fieldset1.UVs (this works)
I wrote the Double_AdvectionRK4 kernel following your suggestion, but it fails with this error:
Traceback (most recent call last):
File "forcoast.py", line 80, in
kernels = pset.Kernel(Dbl_AdvectionRK4_3D) if run3D else pset.Kernel(Dbl_AdvectionRK4)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/particleset.py", line 763, in Kernel
delete_cfiles=delete_cfiles)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/kernel.py", line 148, in init
self.funcvars)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/codegenerator.py", line 391, in generate
self.visit(py_ast)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/ast.py", line 253, in visit
return visitor(node)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/codegenerator.py", line 422, in visit_FunctionDef
self.visit(stmt)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/ast.py", line 253, in visit
return visitor(node)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/codegenerator.py", line 745, in visit_VectorFieldEvalNode
self.visit(node.field)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/ast.py", line 253, in visit
return visitor(node)
File "/data/anaconda3/envs/py3_parcels/lib/python3.6/site-packages/parcels/codegenerator.py", line 720, in visit_VectorFieldNode
self.vector_field_args[node.obj.ccode_name] = node.obj
AttributeError: 'VectorField' object has no attribute 'ccode_name'
Do you have an idea how I should set this attribute ?
Thank you very much !

Hi Luc
I dug a bit into the code this afternoon, because I hoped it would be fairly simple to implement NestedSummedFields. But alas, it's not simple at all I fear.. It would require a major rewrite to do this, and I'm afraid we don't have the resources for that at the moment
As I said, the easiest work-around would be to treat your two fields that you want to sum as completely separate, and only use the NestedFields functionality of Parcels. You'd then need to write a custom kernel for the summing of the fields, but that you can do with the hints I provided above
Good luck!

authors: erikvansebille lucvandenbulcke

repo: OceanParcels/parcels


text:
Adding Kafka as a broker

Any comments are more than welcome!
While testing this, I found that there are update events where the oldobj and newobj are equal, so the data sent contains "duplicates". Do you know why?

Tks for the PR @kaboing.

While testing this, I found that there are update events where the oldobj and newobj are equal, so the data sent contains "duplicates". Do you know why?
It may be due to the Reconcile event that happens overtime even if an object is not updated. This is part of the controller runtime. The best way to test if the are really the same is by checking the "ResourceVersion" fields.

repo: Octops/agones-event-broadcaster


text:
updated the transform origin for menus

Fixes
Issue Number:

What is the current behavior?
Currently, menus have their transform origin in the middle so they expand from the center.

What is the new behavior?
This PR sets the transform origin on the top of the menu.

Other information
Unfortunately, when there isn't enough screen space to display the menu below the target element, the menu will appear on top and it will look like this:

Future Enhancement
It would be nice if it was possible set the transform origin on the bottom if the menu is above the target element
That would look like this:

I couldn't figure out an easy way to determine if the menu was above or below the target element, so I jus set the transform origin to the top in all cases. I think this makes sense because according to the reach ui menu button docs, the menu will always try to position itself below unless there isn't enough screen space. That means in most situations the menu will display below and the transform origin will be correct.
PR Checklist

Toggle...
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[x] Other - Please describe: UX Improvement

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I added a line describing my change to CHANGELOG.md
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

Note: Xander tried this before and we had to revert later.
Not sure if this PR's approach is the same, but whoever's reviewing this should probably dig up what broke in Xander's PR and see if it happens with this approach.

Here's Xander's attempt: https://github.com/lbryio/lbry-desktop/pull/5381
Here's the scenario that requires the revert: https://github.com/lbryio/lbry-desktop/pull/5439

@infinite-persistence  Thanks for linking the PR!
It looks like I took a slightly different approach. I didn't modify the animation at all, only the transform origin. I don't think this approach would recreate that error, but I will try to pull the old issue, recreate it, and see if this PR has that issue.

Yup, agree. From a glance, it looks good :+1:

@infinite-persistence
I think I was able to recreate the old issue. For whatever reason in the old version it would auto click the first item in the menu.

In this PR it should work as expected.

Thanks for the contribution @MaxKotlan, appreciate it! Shoot me an email at tom@odysee.com so we can get some credits over to you for the help and maybe see if you're interested in contributing more :)

@MaxKotlan, do you think it's possible to animate from down-to-up when necessary?  Currently, if the button is near the bottom edge, the popup still flows from top-to-bottom.
Don't worry about it if there's no simple fix.

@infinite-persistence I think it is possible, but it would require a bit of effort.
We would have to figure out how to hook into this position property on the menu/popover.
https://reach.tech/menu-button#menupopover-position
If we can do that, all we have to do is swap out a css class to change the transform origin.
We would probably have to update all the menus individually, or create a wrapper component and update all the menus to use the wrapper component.

Thanks for checking :+1: .  Yeah, probably not worth the effort for now.

repo: OdyseeTeam/odysee-frontend


text:
Fix configuration for local build

Updated scripts/config.ts to be able to run the test node against local nitro code.

hello!
local builds are not done with release branch but with the master one. thank you!

authors: nomaxg tsahee

repo: OffchainLabs/nitro-testnode


text:
ANNOUNCEMENT: Issues approving PAM requests through PAM UX

There is currently an ongoing issue where Microsoft Graph data connect requests approved through the PAM UX will ultimately result in a pipeline failure. As a workaround, use the Exchange Online PowerShell module to approve requests. We will continue to provide updates as we work toward a resolution.

This issue is resolved

repo: OfficeDev/MS-Graph-Data-Connect


text:
Fixed Bot-Proactive-Messaging-Teamsfx Lifecycle - Provision issues

Issues Link -
https://github.com/OfficeDev/Microsoft-Teams-Samples/issues/1311

Build Pipeline -
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/11549127695

repo: OfficeDev/Microsoft-Teams-Samples


text:
CommandBar first item of CommandBar-mainArea become hidden

NavBar was replaced with CommandBar as per your suggestion: https://github.com/OfficeDev/Office-UI-Fabric/issues/385
It does looks more robust than NavBar, but still has an issues I would like to bring up.

I know my commands are not going to overflow. So I have removed   section completely. This will lead to incorrect processing of Commands inside 'Jquery.CommandBar.js'. The function "processCommands" uses "overflowwidth" parameter directly without any check. When HTML node removed the value will be "undefined" and you add this directly to "rightOffset". You probably should have something like this before use of the parameter ...
if (typeof overflowwidth === "undefined") {  overflowwidth = 0; }
I work around this issue by keeping the HTML tag.

I don't need  entire "ms-CommandBarSearch" item and I have removed it. The content of "ms-CommandBar-mainArea" has "padding-left:58px" and command bar icons starts from this position. When I override "padding-left:8px" (8px is the padding-right for "ms-CommandBar-sideCommands") first item in the "ms-CommandBar-mainArea" disappear. How is it happen ...
`    // my parameters as follow: (commands, 8, 0)
// rightOffset will be calculated to 10
var processCommands = function(commands, width, overflowwidth) {
var overFlowCommands = [];
 for(var i=0; i < commands.length; i++) {
   var $Item = commands[i].jquery;
   var rightOffset = commands[i].rightOffset;

   // If the command is outside the right boundaries add to overflow items
   if(!$Item.hasClass('ms-CommandBarItem-overflow')) {

// my condition will not evaluate properly: (10 + 0) > 8 !!! Item become hidden
        if((rightOffset + overflowwidth) > width) {
          overFlowCommands.push($Item);
        } else {
          // Make sure item is displayed
          $Item.removeClass('is-hidden');
        }
      }
    }
    return overFlowCommands;
}

`
I am working with narrow interface, only icons are displayed.The item gets "is-hidden" class at the first load. After refresh calculation work correctly and icon appear.
I worked around this issue by setting "padding-left:11px" for "ms-CommandBar-mainArea"

The issue with removing the search box and having too much padding is being tracked in #363. I've also added a note there describing your issue with the first command disappearing after you change the padding.
Right now the CommandBar assumes that you'll have the overflow area for commands that don't fit. Even if you aren't expecting to need this, does it cause problems to leave it in? We could make that behavior optional but just need to understand the use case for doing so.

There is no problem to keep it in. In fact as my work around I left the top level node in place as follow ...
                   <!-- Overflow Command -->                     <div class="ms-CommandBarItem ms-CommandBarItem--iconOnly ms-CommandBarItem-overflow">                     </div>                     <!-- End Overflow Command -->
This gives me input parameter overflowwidth set to 0.
As far as I see the points of making change as follow ...

If I don't want to have overflow I should be able to delete the section completely. why would I leave the part of the HTML section in place? Just to make variable equal to zero? You may have this assumption in the control itself and set overflow to 0 when entire section doesn't exist.
If I leave it in place I would probably want to leave it completely, as the part of the control, not just top node as I did. In this case I have to implement "ms-ContextualMenu", set "ms-CommandBarItem-commandText" (I have localization and have to set every single line via code). Why would I want to do all this work to make the section complete?
The last point  ... as the general rule for development if I have something in my HTML I have to take care of it, not relay this part will never shows up. Otherwise I should remove it completely.

I hope this is confirm there is certain use case for this simple control change. Otherwise there is work around anyway.
Thank you for your attention, Mike.

The main remaining issue here seems to be the overflow menu, and wanting to remove it if there are few items in the CommandBar. With the new CommandBar updates the overflow menu only appears when the items do not fit. So you will no longer have to manually add/remove the overflow menu. 😄

repo: OfficeDev/Office-UI-Fabric

license: mit


text:
working on OWA but not loading in desktop client

Hi,
I tried this add in combination with angular app, its working fine as it says but only in OWA. But its not getting loaded in Desktop Client. Is there something which needs to be done in manifest file to make it run in the outlook desktop client.
Thanks

Hi,
The highlighting works in OWA (outlook.com) but does not work in the desktop client. The order number is detected in the desktop client and the add-in bar appears , but the order number is not highlighted. However on the web client the highlight and the add-in bar works fine. Any ideas?
Thanks
Desktop client: No highlight : Mac outlook 16.11

Web client : With Highligth

Hi,
Did you do some changes in the manifest file? Or can you please share the code? Because in my case its only loading on exchange server. It doesn't appear in the add-in bar in desktop client not even in windows. But I can see the add-in manifest file is listed in Manifest folder: C:\Users{username}\AppData\Local\Microsoft\Office\16.0\Wef{96571F64-B6B1-4898-A500-3DC4952F5FFA}\2o0JcQKAvdECaRQnqB8vGQ==\Manifests
Thanks

When I wrote this sample this was only working in the web client, the desktop client had not yet fully implemented this. Let me double-check with the Outlook team to see where they are with this.

Hi @jasonjoh any updates to make the add-in work with Outlook Desktop on windows?

@wamaina can you provide details on expectations for desktop?
@nitslucky It works for me in my Outlook 2016 on Windows, but I'm using pre-release versions :). I don't know the current status of what is publicly released.

I have Version 1710 build 8625.2127. Do I need to update this to a more recent version to make it work?

Thanks

@nitslucky did you ever manage to get this working on the native Outlook application? "DetectedEntity" is only working on OWA for me.

@imovertheinternet no just on OWA.

@nitslucky @r-puri @imovertheinternet Contextual add-ins should now be available in Windows and Mac. See related documentation article.
Let us know if you're still facing issues with this.
Thanks.

authors: ElizabethSamuel-MSFT imovertheinternet jasonjoh nitslucky r-puri


text:
IsSubSite not working when using admin context

Q
A

Bug fix?
Yes

New feature?
No

New sample?
No

Related issues?
No

What's in this Pull Request?
Fix for IsSubSite, previously when using the -admin context, and then opening another site collection, the new site collection would be compared with the -admin root web id when checking to see if it's a sub site, and always returning true.
Reproduce:
string tenantName = "mytenant";
using (ClientContext ctx = new ClientContext($"https://{tenantName}-admin.sharepoint.com"))
{
ctx.Credentials = credentials;
var tenant = new Tenant(ctx);
Site site = tenant.GetSiteByUrl($"https://{tenantName}.sharepoint.com");
Console.WriteLine(site.RootWeb.IsSubSite());
}

Hi @thlbrg, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@thlbrg, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Hi thlbrg,
Looks like a good change but we'll need to first test it before we can merge it. Thx for your contribution.

Hi thlbrg,
Testing showed that this implementation did not work. Can you please cross check and verify this...maybe we've missed something but our IsSubWebTest() test case fails with your implementation.

Interesting, i'm getting all green here, will take a look when I've more time.

authors: jansenbe msftclas thlbrg

repo: OfficeDev/PnP-Sites-Core


text:
Add multi-value lookup and user fields.

Added support for multi-value lookup and user fields to have data
provisioned from the template.

Hi @SiobhanBaynes, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@SiobhanBaynes, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Hi SiobhanBaynes,
Thx for this change, looks good but submitted against the wrong branch. Can you please resubmit this PR against our DEV branch as we can't except PR's coming against our master branch. See https://channel9.msdn.com/blogs/OfficeDevPnP/PnP-Web-Cast-How-to-get-started-with-Office-Dev-PnP for more instructions on how to do this.

Sorry, Bert.  It's my first contribution!
I'll do it again now.
Kind regards,
Siobhan.

On 27 Jun 2016, at 16:52, Bert Jansen notifications@github.com wrote:
Hi SiobhanBaynes,
Thx for this change, looks good but submitted against the wrong branch. Can you please resubmit this PR against our DEV branch as we can't except PR's coming against our master branch. See https://channel9.msdn.com/blogs/OfficeDevPnP/PnP-Web-Cast-How-to-get-started-with-Office-Dev-PnP for more instructions on how to do this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

authors: SiobhanBaynes jansenbe msftclas


text:
AzureDeploy.Json

I am getting this error when trying to deploy. I have verified the json file is there
Deploying app services, Azure function, bot service, and other supporting resources... (this step can take over an hour)
<< was unexpected at this time.
Fetching deployment status to check if deployment really failed...
ERROR: (DeploymentNotFound) Deployment 'azuredeploy' could not be found.
Code: DeploymentNotFound
Message: Deployment 'azuredeploy' could not be found.
ERROR: (DeploymentNotFound) Deployment 'azuredeploy' could not be found.
Code: DeploymentNotFound
Message: Deployment 'azuredeploy' could not be found.
Collecting deployment logs...

Hi @Quson-Team ,
Thanks for raising the query here. We are making a follow up internally

We are closing the issue as we are following internally.


text:
Fix for a message size is too large error

For multi-level multi-turn QnA KB, storing all previous questions in an AC payload for every prompt will cause the message size is rapidly exceeded MSTeams message size limit. Because in the original code all previous questions are never used storing just the last previous question into an AC payload will fix a message size issue and I hope won't cause any breaking changes.
Fix #121
Fix #112

@aosolis please take a look.

repo: OfficeDev/microsoft-teams-apps-faqplus


text:
Bot User display name

Hello.,
I will like to change the display name for the Bot User . Currently we get the cards from Bot User - 2b57bc41-596....and will like it so show as "IceBreaker Virtual Admin" instead.  But, I cannot find any documentation on how to do it.  Is it possible and do you know how?
Thank you!
Corina

Hi Corina!
There are two places you should change the display name to be sure it's uniform across all channels.

Bot resource on Azure (go to settings, you will see a display name property over there)
Inside the manifest's json that is included in the package that should be uploaded to Teams. Here you will also follow find a name property.

If you need further help, please say so! Happy to help you :)

authors: CorinaLancheros Rutger-Knijnenburg


text:
Deployment Script Error: Set-AzKeyVaultSecret

Having an error with the 2 lines of Set-AzKeyVaultSecret on lines 547 and 548 regarding KeyVaults:

"Set-AzKeyVaultSecret -VaultName $KeyVaultName -Name 'appid' -SecretValue (ConvertTo-SecureString -String $global:appId -AsPlainText -Force) | Out-Null"
"Set-AzKeyVaultSecret -VaultName $KeyVaultName -Name 'appsecret' -SecretValue (ConvertTo-SecureString -String $global:appSecret -AsPlainText -Force) | Out-Null"

The same error is thrown:
"Set-AzKeyVaultSecret : Operation returned an invalid status code 'Forbidden'"

Note the 2 warnings prior to the errors. There were no errors prior to this.
Thank you!

Hi @alexc-MSFT , any updates on the issue

@alexc-MSFT If I'm not mistaken, I am a global administrator. I have been able to run these lines without errors before, I'm unsure why they are being thrown now. I have tried deleting the key vault and rerun, as well as just creating an entirely new key vault (with a different name).

@alexc-MSFT I was having some issues with some of my other apps I am trying to deploy onto Teams, turns out I was providing my Azure testing environment tenant ID rather than my Teams' tenant ID (where I am deploying my app). I tried to run the script with my Teams' tenant Id this time around, but I got the following error.

All the other apps I deployed using the GUI on the web. I suspect it is because the script uses the same tenantID to deploy the app as well as create resources. Could this be the issue?


text:
Function name is wrong in documentation

In the Register an event handler section, the text refers to a handleDataChange function:

The following code sample registers an event handler for the onChanged event in the worksheet named Sample. The code specifies that when data changes in that worksheet, the handleDataChange function should run.

But the code in the following example actually uses a function called handleChange:
    worksheet.onChanged.add(handleChange);

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

ID: 52542c44-015a-746d-5d6f-c43f6e4e6774
Version Independent ID: dfe8c9fb-4107-e19b-972a-c5a58eeb2646
Content: Work with Events using the Excel JavaScript API - Office Add-ins
Content Source: docs/excel/excel-add-ins-events.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@jpickardsqr Thanks for raising this issue! The fix should be published next week.

authors: ElizabethSamuel-MSFT jpickardsqr

repo: OfficeDev/office-js-docs-pr


text:
Update 5 test.py

строчка 5 принимай

Это странная фигня.

authors: Nayil-Nord Okb-Nord

repo: Okb-Nord/made-eyse


text:
Rework FileChooserView internal structure

As FileChooserView only provides static factory methods, it cannot be used as an actual view.
Redesign of this class is needed in order to reduce amount of code needed for view construction and in order to have a single place where the FXML is loaded and a suitable Pane (StackPane or AnchorPane) is created.
This somehow should also solve the problem in SwingFileChooser to properly pass in a DirectoryChooser which works modal compared to any underlying Swing Window.

Reworked view class, so that FXML is now loaded by convention and Skin is applied automatically. No static methods exist anymore. The view is basically a pane type, where the constructure cares about FXML loading. The FXML name is determined by class name, followed by ".fxml" extension.
See 74e25f3a71e8aa03c7dd0fd1721abd63bdf8c980.

repo: Oliver-Loeffler/FXFileChooser


text:
Decay ek as property

Add initialization of ek (energy spectrum 'ek') depending on wavenumber 'k' as property

Update for DecayTurb:
Specification of the dimension within the classname is no longer necessary.
The dimension depends now on the stencil's dimension.

repo: Olllom/lettuce


text:
🛑 Helix Intranet API - Dev is down

In c309a89, Helix Intranet API - Dev (https://hbi-intranet-backend.onrender.com/status) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Helix Intranet API - Dev is back up in dcb50da after 19 minutes.

repo: OluwaninsolaAO/uptime-monitoring


text:
Add .nvmrc

Specify the node version required to run the app.
While the docs mention that Node 14 is required, Node 14.17.0 is the minimum version required to run the project, adding .nvmrc allows to pick such a version automatically by nvm.

I didn't touched src/locales/translations, I have no idea how this got committed because I didn't commit the changes.

I didn't touched src/locales/translations, I have no idea how this got committed because I didn't commit the changes.

translations come in on yarn install

authors: AlexITC appleseed-iii

repo: OlympusDAO/olympus-frontend


text:
[Feature Request] Add quick postgre startup script with docker

From kirimase created by corentin-bot: nicoalbanese/kirimase#139
Hey,
Just playing with kirimase 👍
Add I thought it could be great to have a script to start un postgre database directly in docker. I used to dev with T3 stack and they have this feature. It's not a big deal to setup, but it's still nice to just create it with just a script command.
Here is the link of the script for postgre and mysql:
https://github.com/t3-oss/create-t3-app/tree/main/cli/template/extras/start-database
What do you think ?

Hey can you tell me more about this , and what would be my approach for making this work

+1 here.
The way I'd approach it is by adding a docker-compose file when the init script is run.
The compose file will have a container for mysql or Postgres.
Then add a command to the package.json to run the docker-compose command if the database could not be found.

repo: OmegaHawkeye/kirimase


text:
Debug Adapter Server / Client update

This updates to the latest version of the Debug Adapter Protocol bringing in some of the missing pieces, cancellation, progress and a few other things.
Also adds support for a Server / Client pair exactly like Language Server Protocol has, these are super awesome in testing scenarios.

The initialization process is handled.
Progress can be sent from the server to the client.
Cancellation is now supported

This required some gymnastics to get right
The cancel handler (implemented by IServerProgressManager will get essentially empty requests for request cancellation, as they get re-routed down the normal cancellation path as provided by the rest of the json rpc bits.

The data sent from the server to the client includes the LSP error codes, which are used to re-hydrate the proper exception types on the client side.

Minor change to some types (though they are really only used on the the internals) to include a method, this was to properly fix serialization of DAP responses.

cc @TylerLeonhardt @rjmholt

Some things I wanna make sure are addressed are:

How do we extend the launch and attach messages? Currently, we redefine our own type.https://github.com/PowerShell/PowerShellEditorServices/blob/master/src/PowerShellEditorServices/Services/DebugAdapter/Handlers/LaunchAndAttachHandler.cs#L30-L95
How could I disconnect and then later connect to a debug adapter server? Currently, we do this horrible event stuff where we:

Trigger this event In the "disconnect" message
That Disposes of the old server and restarts it (YUCK)

I knew there were pieces I need to make sure got updated, thanks I'll look at this tonight.

@TylerLeonhardt have any thoughts on how to make it so those requests are more flexible?
I can make it a Dictionary or add a [JsonExtensionData] property... however none of those will be "strongly" typed.  Re-defining it honestly seems like the better approach.  Perhaps if you derived from the requests themselves and added the properties you want....  I'm not sure.
I didn't get to the second point today, I'll look at that tomorrow.

👀👀👀👀👀👀

authors: TylerLeonhardt david-driscoll


text:
Installing on Fedora 23

I've already installed the dotnet cli for centos on fedora 23, but when i tried to install the omnisharp-roslyn, it forced the download of an inexistent version for fedora and failed while executing the task BuildEnvironment, throwing the message:

dotnet_install: Error: OS name could not be detected
dotnet-install: Downloading https://dotnetcli.blob.core.windows.net/dotnet/beta/Binaries/1.0.0-preview1-002702/dotnet-dev--x64.1.0.0-preview1-002702.tar.gz

How can I change this URL for the one used on centos version?

dotnet's version installed : "1.0.0-preview1-002702"

Hi Victor, are you using VS Code? Or another editor?
I assume that download url is coming from your editor, right? If you're using VS Code, then you should report this as an issue here: https://github.com/OmniSharp/omnisharp-vscode
Otherwise, I would suggest downloading the CentOS build directly from the releases page: https://github.com/OmniSharp/omnisharp-roslyn/releases
And then following the instructions in the read me https://github.com/OmniSharp/omnisharp-roslyn/blob/dev/README.md#vs-code

thank you @will, and sorry for the late answer.

Glad I could help
On Sat, Nov 19, 2016 at 13:04 Victor Borges notifications@github.com
wrote:

thank you @will https://github.com/will, and sorry for the late answer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-roslyn/issues/577#issuecomment-261739551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAHtYf26JxKiW0yg_rn86TKk0JnDsGKks5q_2R2gaJpZM4Itnv3
.

authors: victor-borges will willl

repo: OmniSharp/omnisharp-roslyn