Fix iframe notification items.

Whoops!
There were 4 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2017/03/chrome-58-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/08/chrome-61-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/09/chrome-62-deprecations.md#L5 - WF Tag wf_updated_on must be within the last 7 days.
src/content/en/updates/2017/09/chrome-62-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2017/03/chrome-58-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/08/chrome-61-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/09/chrome-62-deprecations.md - No wf_blink_components field found in metadata. Add if appropriate.


[Mobile] Inline Code Style - Background color issue

Shown in the screenshot, the inline code seems has two layers - tiffany blue on top and white rounded corner rectangle on bottom.
Below is the code
inlineCode: quill.InlineCodeStyle(
  style: TextStyle(
    color: Theme.of(context).colorScheme.onSecondary, -- white font color
    fontSize: 14,
    backgroundColor: Theme.of(context).colorScheme.secondary, -- tiffany blue
  ),
),

How do I use the bottom layer and change color to the tiffany blue?
also, the backgroundColor property in the InlineCodeStyle isn't doing anything.
Thanks
Flutter 3.10.5
flutter_quill 7.3.3

inlineCode: quill.InlineCodeStyle(   style: TextStyle(     color: Theme.of(context).colorScheme.onSecondary,     fontSize: 14,     backgroundColor: Theme.of(context).colorScheme.secondary,   ),   backgroundColor: Theme.of(context).colorScheme.secondary,  -- no effect ),

Anyone has experienced it before?


Set workspace-base to /drone

As the drone-agent sets it to /drone as well.
https://github.com/drone/drone/blob/076dc0c3b93b1acde22ee68d4f5506f7d6538efd/server/hook.go#L509

good catch


Unable to clear the selected image

Hi,
I can't get this working, once I select the image, I'm trying reset the image whenever modal is closed. Is there any way to reset the cropper to initial state without image src?
I tried to set src to null, but it is not working.

Use this.$refs.cropper.reset()

Hi,
I can't get this working, once I select the image, I'm trying reset the image whenever modal is closed. Is there any way to reset the cropper to initial state without image src?
I tried to set src to null, but it is not working.

How do you setup vue-cropper in modal? can you share sourcecode?


Provide option to reconnect to Jupyter kernel

When running a large ipynb on a remote server, losing connectivity to the kernel (due to network interrupt, or closing/reopening editor) can be hugely disruptive, causing me to need to reload the entire notebook.
Jupyter's web-UI does not experience this same issue, allowing you to reconnect to the same kernel for a given notebook on page refresh
Please provide an option to select a kernel to connect to during jupyter connection instantiation

This is in progress. We should decide if we're okay with @hochshi's implementation or not.

Here's an image of it working:
https://user-images.githubusercontent.com/574349/66588359-f7683680-eb94-11e9-9410-de9fec50e9ba.gif

This is now possible.
When in remote sessions, one can use the kernel picker to select an existing (active) kernel.
This feature will be in the next release.
@rchiodo @greazer /cc


Nodes stop processing after a while

Repro steps

Create a Value node
Create a Combine node
Connect the nodes
Delete the Combine node
Undo and redo the deletion up to 20 times

Result
After a while the Combine node won't get a thumbnail any more.
Note
In fact, all nodes have stopped processing, making the program useless until it's restarted.

Fixed!
The issue was that I was too eager to shut down the program when something went wrong. I fixed this by being more specific in my error handling, and also looking over the order of operations and never assuming that a node exists, since they can be deleted at any time from the outside.


Use gulp for all build steps

All build steps should be handled in the gulp script. Currently version patching of the task and extension are done using a PowerShell script called from AppVeyor which also handles the packaging to a zip files. Ideally AppVeyor just calls a gulp task and publishes the results.

This would require GitVersion as an npm / gulp module: GitTools/GitVersion#656


Avoid HTTP 401 error on manifest.json due to CORS

Because of CORS, the manifest.json request returns an HTTP 401 error.
Explaination: https://github.com/koajs/basic-auth/issues/19

Makes sense. Thanks!


No BLE module found

Having the same issue (https://github.com/bardiabarabadi/uvicMUSE/issues/7#issuecomment-744085039) in Windows 10.
I have checked Radios / Bluetooth access in privacy settings but still the same error.
Any suggestions?

Pinging this as I am having the same issue. I'm using the Sabrent BT-40 adapter recommended on the lab page. I have installed the drivers provided by the manufacturer and my laptop recognizes the device as a bluetooth radio. I disabled the onboard bluetooth radio and I still get this issue.
Suggestions?

Same issue here

did anyone find a solution?

I'm having the same issue. The terminal has permission to access Bluetooth, which solved this issue in "uvicMuse not starting on Osx 11.0.1 #7", but is not fixing it for me.
When I use the python code in that issue, I get an error after the line "M_wrapper.search_and_connect()"
Python 3.8.16 (default, Mar  1 2023, 21:19:10)
[Clang 14.0.6 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.

from uvicmuse.MuseWrapper import MuseWrapper as MW
import asyncio
loop = asyncio.get_event_loop()
M_wrapper = MW (loop = loop, target_name = None, timeout = 10, max_buff_len = 500)
M_wrapper.search_and_connect() # returnes True if the connection is successful
/Users/marks/miniconda3/envs/muse_env3/lib/python3.8/site-packages/uvicmuse/MuseFinder.py:18: FutureWarning: The discover function will removed in a future version, use BleakScanner.discover instead.
devices = await discover(timeout=2)
Traceback (most recent call last):
File "", line 1, in
File "/Users/marks/miniconda3/envs/muse_env3/lib/python3.8/site-packages/uvicmuse/MuseWrapper.py", line 37, in search_and_connect
loop.run_until_complete(mf.search_for_muses(timeout=self.timeout))
File "/Users/marks/miniconda3/envs/muse_env3/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "/Users/marks/miniconda3/envs/muse_env3/lib/python3.8/site-packages/uvicmuse/MuseFinder.py", line 20, in search_for_muses
if MUSE_NAME in d.name and not self._is_already_found(newly_found_muse=d):
TypeError: argument of type 'NoneType' is not iterable

I also installed Scipy since I started with miniconda. The only error I got during install was that it installed cryptography 40.0.2 and pyopenssl 23.0.0 requires a version <40, >=38.0.0. I get the same BLE error with either 40.0.2 and 39.0.2.
Thanks for any help you can provide!

I got this from ChatGPT... and will try it out this weekend.
But posting in case any of these suggestions help anyone else... regardless if I'm successful or not.
The message "no BLE module found" usually indicates that your computer is either not recognizing the Bluetooth Low Energy (BLE) module, or it's not properly installed or functioning. Here are some steps you can take to troubleshoot the issue:

Check your Device Manager: The Device Manager in Windows allows you to see and manage the hardware attached to your computer. To access it, you need to:

Press the Windows Key + X and select Device Manager from the list.
Look for "Bluetooth" in the list of device categories. If you see a yellow exclamation mark, a question mark, or the Bluetooth category is not present, there may be an issue with your Bluetooth hardware or driver.

Check your Bluetooth Driver:

If you found an issue with the Bluetooth category in Device Manager, you should try updating or reinstalling the Bluetooth driver.
To update, right-click on the Bluetooth device in the Device Manager and select "Update Driver". Choose the automatic search option to allow Windows to look for the best driver.
To reinstall, right-click the Bluetooth device, choose "Uninstall device", then restart your computer. Windows should automatically reinstall the driver on startup.
If automatic update or reinstall doesn't work, visit Sabrent's official website and download the latest drivers for your specific model. Install the drivers manually by using the 'Browse my computer for drivers' option in Device Manager.

Make sure Bluetooth is Enabled:

Sometimes, the issue is as simple as having Bluetooth disabled. You can enable Bluetooth by going to Settings > Devices > Bluetooth & other devices. Make sure the toggle for Bluetooth is switched to "On".

Check the Bluetooth Support Service:

Windows runs a service in the background to manage Bluetooth functionality. Make sure this service is running:
Press the Windows key + R to open the Run dialog box.
Type in "services.msc" and press Enter.
Scroll down to find "Bluetooth Support Service".
If the service isn't running, right-click on it and click "Start". If it's already running, try restarting it. To do this, right-click the service and choose "Restart".
Also, ensure that the startup type is set to "Automatic".

Run Windows Troubleshooter:

Windows includes a built-in tool to help troubleshoot and fix issues with various components of your system, including Bluetooth.
Go to Settings > Update & Security > Troubleshoot.
Find "Bluetooth" and click "Run the troubleshooter". Follow the prompts and let Windows find and fix any problems.

If none of these steps work, your Sabrent Bluetooth module may be faulty or incompatible with your system. You may need to contact Sabrent's customer support or consider getting a replacement module.


Fix Checkbox not supporting persistence

When using #[store(storage = "local")] with the Checkbox type the build fails because it does not support serialization.
To check whether it works I just quickly changed the input example and added the #[store] macro, which worked with trunk. I didn't actually check whether persistence worked.
I chose the exact same version/featureset of serde as yewdux itself depends on, hope that is a good idea.

Looks great, thanks!


Unit test

Disclaimer: I made only a few unit test on small projects so far, no with unity modding especially.
I have setup a unit test project for Jotunn, this should be useful in the long run. It uses basic NUnit, not the Unity Test Framework. A few changes had to be made, I removed MonoBehaviour from Manger.cs to properly test these child classes. This should be not be a problem because those managers don't use MonoBehaviour as far I've seen.
I added a few tests to the ObjectManager class as a demonstration (JotunnTest/ObjectManagerTests.cs).
This is only a proposal, it's up to you guys if you want to write unit tests along. I do not intent to maintain this, only write a few test cases.

It is not possible with MonoBehaviour because a class that inherits it cannot be instanitiated outside of Unity. So you cannot call new ObjectManager() for example. And therefore all manager classes cannot be tested.
The only way to keep MonoBehaviour is to use the Unity Test Framework and having a Unity project that inlcudes all necessary dll files.

So, I'm not entirely happy with our implementation of singleton and was thinking about implementing a more generic DI solution which should also make things a little more simple for unit testing, however i'll need to talk with everyone else. If you want to help us contribute to this project and be a little more involved in discussions please add me on discord and i'll add you to the group chat. Thank's for your PR and we'll have a think about how we will resolve some of these issues.
paddy#1337

Thanks for your response, I'm closing the PR now.


Expose active interpreter path via command

Add support for "pythonPath": "${command:activeInterpreter}" to assign the active interpreter path in launch.json.

Closing this issue as ${config:python.pythonPath} already provides this functionality in a more robust way.
On seeing this ${command:activeInterpreter} VSCode executes a command, but that command cannot accept any arguments. Due to this limitation, we don't know which workspace folder to get settings from.
A workaround to this: This command was implemented in a way such that it returns the Active interpreter instead, one that appears in the status bar. But what if the active interpreter doesn't update correctly, this will then select the wrong interpreter to execute files.
${config:python.pythonPath} does not have this problem, as we know the workspace folder there.


[Bug] Incorrect Redirect from "Fern Definition" Link on OpenAPI Overview Page

Describe the Bug
There is a navigation issue on the OpenAPI Overview page under the API Definition section. Currently, when users click on the "Fern Definition" link, they are redirected to the homepage (https://buildwithfern.com/learn/home) instead of the specific page for Fern Definition.

Expected behavior:
The "Fern Definition" link should redirect users to the Fern Definition overview page at https://buildwithfern.com/learn/api-definition/fern/overview.
Steps to Reproduce
1)Go to https://buildwithfern.com/learn/api-definition/openapi/overview.
2)Click on "Fern Definition" link.
3)Observe the redirection to the homepage rather than the expected Fern Definition overview page.
CLI Version
version: 0.23.1-rc6
Proposed solution:
Please update the "Fern Definition" link to correctly point to https://buildwithfern.com/learn/api-definition/fern/overview.
This adjustment will improve navigation and user experience, ensuring that users can easily access the correct documentation for Fern Definition.

Please assign this to me, i got the solution

hey @abvthecity are you there??

Hey @last-developer, thanks for the fix! feel free to tag @dsinghvi in the future


works not well with sniproxy

(Are you asking for help with using Caddy? Please use our forum instead: https://forum.caddyserver.com. If you are filing a bug report, please answer the following questions. If your issue is not a bug report, you do not need to use this template. Either way, please consider donating if we've helped you. Thanks!)
1. What version of Caddy are you running (caddy -version)?
0.9.5
2. What are you trying to do?
reuse port 443
( run softhether & ocserv & caddy on a single vps, all of them use 443 by default,
which sniproxy can handle them with different domain name )
nginx works properly but it's troublesome to use Let's Encrypt's certificate
3. What is your entire Caddyfile?
my.domain:444 {
  header / -Server
  root /var/www
  redir {
    if {scheme} not https
    to https://my.domain/
  }
}

www.my.domain:444 {
  header / -Server
  redir https://my.domain/
}

4.  What did you expect to see?
5.1 https://www.my.domain works well with sniproxy
5.2 visit http://www.my.domain and return https://www.my.domain/
6. What did you see instead (give full error messages and/or log)?
6.1 https://www.my.domain works well with sniproxy【✓】
6.2 visit http://www.my.domain and return https://www.my.domain:444

Are you sure you want to redirect http://www.my.domain to  https://www.my.domain/? From your redir https://my.domain/ line, it looks like you want to redirect http://www.my.domain to https://my.domain/.

Closing due to inactivity...


Extract WaitForStream function

The wait code from the example app is quite helpful -- This PR give users of
the library an opportunity to leverage it without duplicating into their own libraries.

Extract WaitForStream function

Minor changes

Add short-circuit on CreateStream if it already exists
Move the Get records to bottom so it fetches all the records
Update the batch put so the records are distinguishable from single puts

Don't forget rebase from master :)

Hi. Any plans on getting this merged? Thank you.

Hello, @dlsniper . I wrote in comment, what better to change for this pull request. I think @harlow can be busy to make needed changes.

Ahh thanks for the bump. Yes let me rebase master and address the comments.

@le0pard could you please elaborate on these?

Let's make StreamExists call optional by additional parameter (for example boolean) in method CreateStream.

To my knowledge golang doesn't allow optional parameters. Could you give me an example of what this might look like?

Need configure inside WaitForStream how long we want to wait and max retries: again configurable by params timeout and 4 second timeout (should be default if not set)

Is your thought that the config would be part of the kinesis struct?
func (kinesis *Kinesis) WaitForStream(StreamName string) {
  # ...
			time.Sleep(kinesis.WaitForStreamSeconds * time.Second)
  # ...
}

To my knowledge golang doesn't allow optional parameters. Could you give me an example of what this might look like?

I think most simplest way - have this separate function, which will wait (you already have this), not need to add this in CreateStream function.

Is your thought that the config would be part of the kinesis struct?

Yes, good idea. Let it be in structure.


Add badges to README.md

Add badges for Travis CI (and any other services used) to the README.

Added a commit which shows what this might look like.

This would be quite useful alongside code coverage metrics. Perhaps hold off until we have gh-142 done and we can add that badge to the readme too.

Merged into develop.


Don't try to send an email to a user without an email

On hold because of bugs in da itself.

Almost ready - accidental early merge needs to be tested. #86

Fixed by #50 and tests in #86


Honojs shows 404 Not Found in Postman on app.delete() route with sqlite db

I have used the url in Postman is http://localhost:3000/user/1 on my local bun server.
In case id 1 already exists in database.
My Code:
app.delete("/user/:id", async (c)=> {
const id  = await c.req.param('id');
console.log(id);
db.run("DELETE FROM users WHERE id=?",id);
return c.json({message: 'Deleted'});
});
but other routes are working well.

Hi @techwebdocs
This error occurred by this bug of Bun.
https://github.com/oven-sh/bun/issues/677
The only one thing that we can do is wait to fix the bug.

hey! you can move the delete verb to the url in the middletime
app.delete("/user/:id/delete", async (c)=> {
const id = await c.req.param('id');
console.log(id);
db.run("DELETE FROM users WHERE id=?",id);
return c.json({message: 'Deleted'});
});

This seems to be closed.


Package build fails on windows due to encoding error

Error for an 'sdist' invocation looks like:
Traceback (most recent call last):
  File "setup.py", line 23, in <module>
    readme = f.read()
  File "C:\Programming\python\stix\venv\maec2stix_py3\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 725: character maps to <undefined>

The error is caused by utf-8 encoding in README.rst.  No encoding is specified in setup.py when the file is opened.  On Windows it defaults to cp1252, which fails to decode the file.  The problem characters are curly quotes around "best effort".

@chisholm,
I'm confused why this should have an impact on the maec converter - does it use the elevator??

Yes, the "stepper".  MAEC 5 has STIX 2.0 SCOs; they needed to be upgraded to 2.1.  Chris pointed me to the work you did for that.

Just pushed a fix to master.


fix empty category name

I checked the app by all means and didn't see any example where it's handled empty category name.
Now on add new category or update existing category the category can't be created or will not be updated with an empty string.
Moreover, applying trim() twice in the same block, why not use it once.

@all-contributors please add @Mudassar045 for code


Add full search instead absolute word

Is your feature request related to a problem? Please describe.
I think is not a problem in the search engine. It is just a minimal improvement.
According to my tests in the demo page: https://pmarsceill.github.io/just-the-docs/ ,in the customization page there is a text:
{% include css/just-the-docs.scss.liquid color_scheme="foo" %}

And if I search with a part of this string : liquid, the search result does not show the customization page

But if I search the entire string:
css/just-the-docs.scss.liquid

Search result is able to find the customization page:

This is why I think, a full search would be helpful instead of to remember the exact text.
Here some posts of people  with the same improvement idea:

https://nachtimwald.com/2020/06/10/full-text-search-with-jekyll/
https://10consulting.com/2013/03/06/jekyll-and-lunr-js-static-websites-with-powerful-full-text-search-using-javascript/

Describe the solution you'd like
Maybe a parameters in _config.yml to allow a full search will be nice:
allowFullSearch: true

@jrichardsz The search with lunr.js supports wildcards. Searching for '*liquid' includes the customization page in the results:

Yes, with wildcard its works.
Do you think some tips related to search commands could be included in https://pmarsceill.github.io/just-the-docs/docs/search/ ?
Thank you for your time.

@jrichardsz Currently, the theme docs provide little or no guidance about the various frameworks that website developers may need to use (Jekyll, GitHub Pages, Markdown/kramdown, (S)CSS, Liquid, JavaScript, etc.). Some tips can be found by searching the responses to (open and closed) issues, and in the Q&A section of the Discussions. The same goes for search: the theme docs page covers only the configuration options supported by the theme, linking to the Lunr framework home page for further documentation.
It might well be useful (especially for users who are new to Jekyll) to have a systematic collection of tips in a new section of our docs. However, somebody would need to volunteer to create that section, and keep it up-to-date...

I can update the readme, just point me to the exact location :D .
Some teams don't have ruby or jekyll skills (i'm java), and they just want a wiki for their markdown documents.

I've been using Jekyll for a few years now for simple static websites. I think most docs for Jekyll at least are more expansive in their own website. I encountered the same barriers of entry when I started, I'll take a look at the docs and see what I can do, if that's ok.

Thanks for your comments regarding the difficulty of getting started with using this theme, and especially for offering to help with improving our docs!
The only other Jekyll theme I've ever used is Minima, and I think the Minima home page assumes one already knows quite a lot about Jekyll. The explanations on the Jekyll Themes page may be more helpful for users who aren't already familiar with Ruby gems and Jekyll themes.
Unfortunately the Jekyll Quickstart page can be confusing for new JtD users, and navigating through the other Jekyll docs pages to find the relevant information can be quite challenging... I think it would be good to add a page to the JtD docs with a guide for new Jekyll users about how to get started with JtD, giving more details than we currently have on the JtD home page. It could include sections (or links to further pages) with tips about using Markdown, kramdown, Jekyll, and Liquid.
I'll be happy to review and approve PRs for such new docs, and merge them into the current master branch, independently of releases. That should be easier to manage than a collection of potentially conflicting PRs for inclusion in a new version of the theme, and it will allow the new docs to appear sooner on the JtD website. (A drawback is that the docs pages will no longer be completely stable between releases, but I think that is outweighed by the benefits.)
If you agree with the above (and @pmarsceill doesn't raise objections to the proposal) I'll post a summary of the ideas in the JtD Discussions then we can follow up there.
In case it helps, the Just the Docs Tests site lists all the steps it took to create it, including the use of GitHub Actions. Caveat: It was using Jekyll-4.1, and it needs updating to account for the new treatment of site.url in Jekyll-4.2.0, which affects local browsing.


Assertion error if hidden list contains a word with 4 or more equal characters

I was trying to run this with a word list that includes Roman numerals, some of which contain four equal letters, e.g. xxxix. This causes wordle to crash with an assertion error:
$ echo xxxix >test
$ ./wordle -h test
Commit 8fd3f1bdf884 2024-03-06 00:27:32 +0000
Initialising...
wordle: wordle.cpp:390: void inithardmodebitvectors(): Assertion `mult[a]<4' failed.

Yes, sorry about that unfriendly error message. It uses an internal coding of two bits per letter in order to pack the letter counts into 64 bits (262<=64), so it can handle letters that occur up to three times. This works for the original wordlist as that doesn't contain any words with >=4 of the same letter, and it just rejects wordlists with >=4 of the same letter. It wouldn't be hard to adapt this to cater for some letters (up to 12=64-262) that occur >=4 times and still pack this into 64 bits. Or could take a slight efficiency hit and go to 128 bits for the counts.


somewhere that needs to be modified

def get_parallel_decay_mask(self, length):
Tensor created in above function is default created on cpu.  If train on cuda, it will throw error like below:
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

You are correct. The recent commit (48a3984008db3daaf564fbd748006517aeb660ee) should solve this issue. Thanks!


OClassLoaderHelper uses javax.imageio.spi.ServiceRegistry

Seems odd that its depending on javax.imageio libraries here.
Suggest you guys change to use java.util.ServiceLocator instead, which IIUC is essentially the same thing you need here.

yes, i also wondered a few times why was imageio there, i'll change this with (java.util.ServiceLoader)[https://docs.oracle.com/javase/6/docs/api/java/util/ServiceLoader.html] after we released 2.0 final.
the reason for not do it now is that we don't want to mess up 2 days before release, and this may impact some osgi environment.

FYI, using imageio impl may actually prevent proper usage in osgi:
http://stackoverflow.com/questions/12664287/does-javax-imageio-spi-serviceregistry-work-in-an-osgi-container

@tglman WDYT?

FTR I can't say if changing to the java.util.ServiceLoader will actually resolve this yet.  I may run some local tests, as it may still be problematic due to how the classloader is selected by consumers of OClassLoaderHelper.  Will let you guys know what we find.

Update: I just tested switching to java.util.ServiceLoader and by itself, it doesn't change the behavior for the issue we're seeing (karaf still can't find OGraphFunctionFactory, so graph functions show up as missing when you try to use them in select statements).

thanks @cwilper, we'll have to dig a bit more to see if there is an issue with how we have the CL setup or if its a more fundamental issue of how orient is loading these based on the loading with specific CL orientClassLoader  = OSQLEngine.class.getClassLoader();

Thanks guys, we really appreciate your work on this.

FYI, a quick solution to this is to make the graphdb bundle a fragment of the core bundle:
diff --git a/graphdb/pom.xml b/graphdb/pom.xml
index 8f4d342..52b76c6 100644
--- a/graphdb/pom.xml
+++ b/graphdb/pom.xml
@@ -40,6 +40,9 @@
         <javac.src.version>1.6</javac.src.version>
         <javac.target.version>1.6</javac.target.version>
         <jar.manifest.mainclass>com.orientechnologies.orient.server.OServerMain</jar.manifest.mainclass>
+        <osgi.fragment.host>
+            com.orientechnologies.orientdb-core
+        </osgi.fragment.host>
         <osgi.import>
             com.tinkerpop.blueprints;resolution:=optional,
             com.tinkerpop.gremlin.groovy.jsr223;resolution:=optional,

This will merge their contents at runtime in OSGi so that they effectively share the same classpath and should therefore make the graph functions visible.

I'm not an OSGi expert. Everybody agrees on this change?

fixed closing


Setup global data listeners for all data except semesters

Depends on #275.
Summary
This PR setups up all firestore listeners except semesters and then connect them to the root Dashboard. Semester data is hard to setup listener due to drag and drop mess.
For all the listeners I set up, when all of the listeners finish the first onSnapshot, it will call the onLoad callback to inform the application that the first batch of data has been loaded (so that the application has full data to work with, like requirement computation). Dashboard turns this into a promise and combines this with the promise of semester data loading, and then wait on these promises before declaring the page is loaded.
Test Plan

add/edit/delete course still works.
course drag and drop still works
courses in the requirement menu still correctly show up

Notes
To integrate semester into this, we need to replace the drag and drop library first.

Thanks Sam! Everything looks mostly good except a couple comments and a bug I noticed:

Req bar updates seem kind of buggy when I drag a course from the req bar to the semesterview. Though I'm not sure if this is because we don't have a global data listener for semesters.
PR #275 staging site. The req bar does update on drop:

Screen.Recording.2021-02-11.at.11.41.36.AM.mov
This PR staging site. The req bar doesn't update on drop. Also, the small compact card is of the crosslisted course, not the one that was dragged. However, the small compact card does disappear when you refresh the page.:
Screen.Recording.2021-02-11.at.11.48.23.AM.mov
Once the comments are addressed, the PR should be good to go!

the bad behavior of the double card is also on master :(, although i'm not sure when it's there. It's probably introduced by me at some point... I'm relatively confident that migrating to a new drag and drop library can fix that.


[marked] add missing renderer? option on marked

Also the fileformat is dos, with CRLF/^M everywhere, which is also weird. should it be utf-8 / unix ?

@Carreau thanks mate!

note
https://github.com/chjj/marked/blob/49b7eaca76d14ccd91f87decc11bbe213a6e0af4/lib/marked.js#L524

Thanks !


FR: Share picture

Any particular reason you can't share a picture? That feels like the most common thing to share.

If we could have a mechanism to share a files/document/picture that would be great.

Can you add support for sharing images, please? Sharing by image file path or by stream will be perfect.
You just need a little to expand the existing code...
Code for Android
public void ShareImage(string title, string text, string imagePath)
{
    var uri = (Android.Net.Uri.FromFile(new Java.IO.File(imagePath)));
    var sharingIntent = new Intent(Intent.ActionSend);
    sharingIntent.SetType("image/png");
    sharingIntent.SetFlags(ActivityFlags.GrantReadUriPermission);
    sharingIntent.PutExtra(Intent.ExtraText, text);
    sharingIntent.PutExtra(Intent.ExtraStream, uri);
    var chooserIntent = Intent.CreateChooser(sharingIntent, title);
    chooserIntent.SetFlags(ActivityFlags.NewTask);
    Android.App.Application.Context.StartActivity(chooserIntent);
}

Code for UWP
List<IStorageItem> imageItems = new List<IStorageItem>();
imageItems.Add(file);
request.Data.SetStorageItems(imageItems);
RandomAccessStreamReference imageStreamRef = RandomAccessStreamReference.CreateFromFile(file);
request.Data.Properties.Thumbnail = imageStreamRef;
request.Data.SetBitmap(imageStreamRef);

@PetrVobornik  could you send that code as PRs :)

Any update on this ?? I'm trying to get this option : SAVE IN FILES

Isnt this repo doing basically the same job but it is extended with pdf and image also?
https://github.com/adamped/ShareDialog/tree/master/ShareDialog
According to this, it doesnt seem to be that hard to implement it.

Any news ?

No new functionality will be added to this library and all are moving to https://github.com/xamarin/Essentials


Support (EXPLAIN SELECT ...) as a subquery (ported from clickhouse #40630)

…0630) (#5857)

Support (EXPLAIN SELECT ...) as a subquery (ported from clickhouse)

minor adjustments and changes

change stateless test file to pass the tests

adjust format and add extra info

small adjustments of format

PR checklist:

Did you run ClangFormat ? Yes
Did you separate headers to a different section in existing community code base ? Yes
Did you surround proton: starts/ends for new code in existing community code base ? Yes

Please write user-readable short description of the changes:
Porting

[x] clickhouse/clickhouse#40630
to close
[x] timeplus-io/proton#819

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

after sign CLA need repush a commit or add a tag to trigger.


401 Unauthorized on new Bot

Bot Info

SDK Platform: .NET
Active Channels: MS Teams
Deployment Environment: Azure, local development with Emulator

Issue Description
Trying to create a new chatbot, but keep getting 401 Unauthorized. Here's the sequence of facts:

I already had some bots created (from when the bot service was still on preview) and had no trouble upgrading;
Created a new one through Azure and deployed (local git repo) the same code that was already working in one of the bots and locally;
When testing it through Web chat on Azure it never answered me, got to a 401 error after analyzing;
Updated all Nuget Packages and starting retesting it locally - started getting 401 error locally now as well

I've already gone through this troubleshooting: https://docs.microsoft.com/pt-br/azure/bot-service/bot-service-troubleshoot-authentication-problems?view=azure-bot-service-3.0
My problem is rising on Step 3: I'm enabling security, but still get 401. Anyone knows what might be the problem?

Did make sure you input your appID and password on both the emulator and in your config file?  Would you mind sharing your code by putting it in a repo so we can take a look?

Are you using LUIS?

@JasonSowers  I did make sure to put the appID and password on both! And unfortunately I can't share the code because it's for my job =/
@ankitbko Yes, I'm using LUIS! (Please tell me it means you know what's wrong haha) It works fine on the other bot though

@robertobandeira haha.. could you check what region you have created your LUIS subscription(not the bot) in?
To check this go to your LUIS application -> Publish -> Resources and Keys (scroll to bottom)  and check what Region and Endpoint domain you have for your published key. It should match your domain in LuisService in bot.

I'm not able to send the image right now, but they do match, both are southcentralus.

Is 401 arising from MessageController or somewhere internally in the bot? You could find it by remote debugging your deployed application.
Just want to make it sure again, you did specify domain in your C# code in LUIS attribute?

In the code I specify it as [LuisModel("xxxxx", "xxxxx", domain: "southcentralus.api.cognitive.microsoft.com")]
This is what I get when I try to start a conversation with the bot locally (just connect with the bot):
{   "type": "conversationUpdate",   "membersAdded": [     {       "id": "default-user",       "name": "User"     }   ],   "id": "a0clm1k754k",   "channelId": "emulator",   "timestamp": "2018-05-10T17:28:27.256Z",   "localTimestamp": "2018-05-10T14:28:27-03:00",   "recipient": {     "id": "n8lfke71a7h6",     "name": "Bot"   },   "conversation": {     "id": "h4737n377eid"   },   "serviceUrl": "http://localhost:57189" }

The error I see on the bot emulator is "[14:28:27] -> POST Response code 401 (Unauthorized) "
The error I get from the version deployed in Azure is:
Message: POST to marketquotes-homologacao failed: POST to the bot's endpoint failed with HTTP status 401
Call Stack: Microsoft.Bot.Schema.BotException:  at Microsoft.Bot.ChannelConnector.BotAPI+<PostActivityToBotAsync>d__30.MoveNext (Microsoft.Bot.ChannelConnector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null) Inner exception System.Exception handled at Microsoft.Bot.ChannelConnector.BotAPI+<PostActivityToBotAsync>d__30.MoveNext:  at Microsoft.Bot.ChannelConnector.BotAPI+<PostActivityToBotAsync>d__30.MoveNext (Microsoft.Bot.ChannelConnector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null)
Response Body: Failed authenticating incoming request: System.IO.FileLoadException: Could not load file or assembly 'Microsoft.IdentityModel.Protocols, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'Microsoft.IdentityModel.Protocols, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'
at Microsoft.Bot.Connector.JwtTokenExtractor..ctor(TokenValidationParameters tokenValidationParameters, String metadataUrl, String[] allowedSigningAlgorithms, EndorsementsValidator validator)
at Microsoft.Bot.Connector.BotAuthenticator.d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Bot.Connector.BotAuthenticator.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Bot.Connector.BotAuthentication.d__28.MoveNext()

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

Do you have it deployed on Azure, and if so is it working there?  I'm asking to see if this is an emulator only issue.  It was a bit unclear in your original post.

It is deployed on Azure and it does not work there. On my last comment I posted the error I get on the version deployed there when I try to test it using Web Chat.
Sorry if it's not very clear. I'm trying to select what information exactly might be important to debug it, but I honestly have no idea even where to search for the problem right now.

Were you by any chance using a trial subscription and now not using a trial subscription?

@robertobandeira I see you've updated all nuget packages.  There might be a conflict somewhere.  Can you share your packages.json, binding redirects in web.config, and/or your project file?  (I'd try downgrading the packages to what they were previously, then upgrade one at a time until you determine which package upgrade broke authentication)

@EricDahlvang The code wasn't working on the Amazon-deployed version even before I updated the packages, it was the local version that stopped working after the updates =/
but I'll try that, it might give me some insight, thanks for the tip!

No errors/stack trace when I run locally. Gonna try adding that redirect and see what happens, thanks a lot! =)

Hey guys, forgot to update the thread, but it worked after I downgraded "Microsoft.IdentityModel.Protocols" to version 2.1.4, both locally and Azure-deployed. Thank you all for the help!


HEAD method in Ajax.Request.

Why is the HEAD method excluded from Ajax.Request?
It should be a standard method supported by all browsers.
Correct me if I'm wrong.

Duplicate of #280.


Wrong connecting to pages whose URL contains special UTF-8 characters

The method Jsoup.connect() in Jsoup 1.10.2 returns a wrong Document when its argument contains Polish UTF-8 characters (e.g. "https://www.google.pl/search?q=gąska"). There is no such a bug in Jsoup 1.10.1.

This is fixed in 1.10.3 by 56a728d482ce0c88cd9513eca1e7c7585f0718f9
Will close this when released. Sorry about the issue!

jsoup 1.10.3 is out now: https://jsoup.org/news/release-1.10.3


When I added calendar list ,schedule can't get calendar ID.

I want to manual add calendar list , but schedule list can't get calendar ID.
one btn
two input (name,color)

$('#addListBtn').click(async function () {
    let calendarName = $('#addListInput').val()
    let calendarColor = $('#listColor').val()
    calendar = new CalendarInfo();
    calendar.id = Date.now();
    calendar.name = calendarName;
    calendar.color = '#ffffff';
    calendar.bgColor = calendarColor;
    calendar.dragBgColor = calendarColor;
    calendar.borderColor = calendarColor;
    addCalendar(calendar);
    console.log(calendar.id + 'add calendar id');
    $('#calendarList').append(
       https://github.com/nhn/tui.calendar/blob/master/examples/js/app.js#L446 // html can't not chow
    )
    $('.dropdown-menu-title[data-action="toggle-monthly"]').click() // rerender calendar
}

Cal event handlers also can't get calendar list ID

  'beforeCreateSchedule': function (e) {       //建立新的scedule
      console.log('beforeCreateSchedule', e);   // e.calendarId = null
      saveNewSchedule(e);
  },

But when I put new calendar list to database and create new page，
rerender calendar list from database , schedule can get calendar list id
How can I let schedule get calendar list Id after I create new calendar list?

@LinX9581
I can nearly get the idea you cannot get the calendarId from the schedule after creating a calendar instance dynamically.
Basically, the beforeCreateSchedule event doesn't have Schedule instance so you cannot get the calendarId from it.
However, the behavior gets changed when you enable the useCreationPopup option. After clicking the 'Save' button on the schedule creation popup, you can get calendarId from the event object.
  const cal = new Calendar('#app', {
    scheduleView: true,
    useCreationPopup: true
  });

  cal.on('beforeCreateSchedule', e => {
    console.log(e.calendarId); // will be null if you didn't choose a specific calendar
  });

Schedule list can't get calendar ID because of "calendar.id = Date.now();"
It should be "calendar.id = String(Date.now());"

@LinX9581 Good news! I've just published the new version resolving the issue. You can use the latest version(v1.13.1) to use number-based calendar ids.


Update documentation for Proxy IP addresses and ports

Description
This PR explains which ports need to be open on which IPs, and adds commands to check that the machine is accepting TCP connections over those ports.
Does not check UDP.
Tested
Not tested
Other changes
None
Related issues

@nambrot please let me know what I'm missing for the attestations service

For the attestation machine, it would be port 30303 as well on udp/tcp for discovery of the node, and then port 80 for the service itself


Posting PropertyVM to MVC controller

Hello,
All the example has it render propertyVM on MVC view, did some try to post the data from MVC to controller with the same PropertyVM object?

Don't modelbind to the PropertyVm, use a JToken - see https://github.com/mcintyre321/FormFactory/issues/54#issuecomment-363841903.


docs: Improve wording on stating that a method does not work in-place

Describe the bug
For methods that return a copy, rather than working in-place, I believe the standard wording in docstrings is:
"""
Returns a new table that/where/with [...].

[...]
"""

Currently, we have this sentence in most of our docstrings:
"""
[...]
This table is not modified.
[...]
"""

Might be unclear for some users what "this table" refers to.
This rule should also be added to the guidelines.

Certainly wrong is this:
    def tag_columns(self, target_name: str, feature_names: list[str] | None = None) -> TaggedTable:
        """
        Mark the columns of the table as target column or feature columns. The original table is not modified.

        This table is not modified.
        [...]
        """

Expected behavior
Agree on a standard wording to use everywhere.
Definitely fix the redundancy in tag_columns in src/safeds/data/tabular/containers/_table.py.

This rule should also be added to the guidelines.

This part of the issue will be resolved by #427.

Blocked by #58 (which will add a bunch of new methods / docstrings in the TaggedTable class).

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

v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


Memory Leak + Auto Restart?

The memory keeps getting higher and higher, so I suggest to obtain a auto restart function. Because I run a 24/7 server, and I get alot of traffic from Wun Wun's video on my server.
Another server client I used was able to be reset by pm2 by restarting? I'm not completely sure.
I manually reset the server after a day.

even this is not the best practice, but you can use the garbage collector in JxCore, enabling it after 24hour, using the flags.

You can write a bash script or cronjob to auto-restart your server.

I started test server 1.2.18 almost 3 days ago (2 days and 9 hours) and it still works with no any memory leak.
Did you used latest version?
Did you modded it?
Try to update to the latest version, it has no memory leaks.
Restart is better with standard functionality (bash script, cron, etc).

My server also randomly goes down so often.  I wrote this code to work around the server going down too often:
#/bin/sh
while true; do node ./index.js; sleep 2; done

I added sleep 2 to give me time to press ^C to break out of the while loop.
Drop this in the /src/ folder as a shell script and run it away!

SimonOrJ: what error happens? Is there any record about error in the log?

I'm not getting any error on the log; it just says "killed" on the console screen.  It might be the mod, but I'm trying to update the server with the latest code right now.

it's strange, because at the moment, test server running for 3 days, 23 hours online. With average 4-15 players online. And there is no memory leaks and crash

I finally got a random error written on the console.
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at exports._errnoException (util.js:870:11)
    at TCP.onread (net.js:552:26)

yeah, I catch the same :)
Probably someone using exploit in ws code to shut down the server

It is a Node bug and is caused by idle sockets.
See issue: https://github.com/nodejs/node/issues/3595
Probably upgrading to latest version of Node should solve the problem.

Ooh, it is caused by recent addition of httpServer.
Need to catch httpServer errors:
this.httpServer.on('error', function (err) {
    ...
});

looks like Node bug

Not sure if this is relevent, but someone posted this on Cigar.

tested httpServer, his error is handled in WebSocket, it pass error to onServerSocketError handler.
So, probably it's not the case.

MastaCoder: it's not relevant, MultiOgar has validation for incoming packet and just will ignore incorrect packets
Tried to reset connections with tcpkill. It's processed by server with no error.
So, it seems like some node.js bug within net.js. Related to async processing.
And someone exploit it to crash the server

I got another error on the console:
FATAL ERROR: node::StreamWrap::DoAlloc(size_t, uv_buf_t*, void*) Out Of Memory
Killed

My server has a memory leak.  I can also see it on htop.  The RAM usage slowly goes up.

what node version you're using?

4.4.6

SimonOrJ: it seems that your server very popular :)
Try to disable websocket compression:
GameServer.js
    var wsOptions = {
        server: this.httpServer,
        perMessageDeflate: true
    };

Set perMessageDeflate to false
Let me know if it reduce lags.

@Barbosik The server's actually at same number of players.  I don't know why it's lagging all of the sudden.

There was a socket leak bug, it keep closed socket in CLOSE_WAIT state forever.
And when it reach max limit it fails.
I fixed it. Hope it was the reason of ECONRESET crash

SimonOrJ: just monitored your server, it has high lag peaks, the pause between packets from server sometimes is too high. Probably it happens when virus hit into some big cell. May be something related with collision (too many collision checks)

@SimonOrJ: here some finding:
PlayerCell.prototype.updateRemerge = function () {
    var age = this.getAge(this.gameServer.getTick());
    // 20: full cell split; 13: cell split delay.
    if (age > 20 || (age > 13 && this.getMass() < 1000))
        this._canRemerge = true;
}

it set _canRemerge = true once and never reset it.
It will be better to replace it with this (check for age < 15 should improve performance, because it reduce collision check):
PlayerCell.prototype.updateRemerge = function () {
    var age = this.getAge(this.gameServer.getTick());
    if (age < 15) {
        // do not remerge if cell age is smaller than 15 ticks
        this._canRemerge = false;
        return;
    }
    this._canRemerge = age > 20 || (age > 13 && this.getMass() < 1000);
}

Before updating to the above code, my node server used over 1.7G of RAM.  This is weird.
I updated the code.


luci-app-advanced-reboot: add support for Linksys WHW03 v2

Add support for Linksys WHW03 V2 (Velop)
The device was merged in openwrt a couple of days ago.
This patch was fully tested on a running device.
Signed-off-by: Vincent Tremblay<EMAIL_ADDRESS>

Thankn you @vtremblay !


i18n(ko-KR): update sst.mdx

Description (required)

update sst.mdx
#9475

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/deploy/sst.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Fix layer loading indicator in tree

Create deep clone from state value before manipulating it.
Extracted from #1775.
Please review @terrestris/devs.

Thanks for the review!


[Breaking change]: Deprecating Desktop Windows/macOS/Linux MonoVM Runtime Nuget Packages

Description
The following desktop MonoVM runtime nuget packages are obsolete starting in .NET 9 Preview 7.

https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.win-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.win-x86
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.linux-arm
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.linux-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.linux-arm64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.linux-musl-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.osx-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.osx-arm64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.LLVM.AOT.linux-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.LLVM.AOT.osx-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.LLVM.linux-arm64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.LLVM.linux-x64
https://www.nuget.org/packages/Microsoft.NETCore.App.Runtime.Mono.LLVM.osx-x64

These nuget packages could optionally be used during .NET SDK publish for self-contained applications via an undocumented SDK switch.  These desktop MonoVM runtime packages are not the default configuration for .NET publish scenarios.  .NET 9 Preview 6 is the last release of these packages.
Version
.NET 9 Preview 7
Previous behavior
During .NET SDK publish, these desktop MonoVM nuget runtime packages were available for self-contained applications using an undocumented SDK switch.
New behavior
Starting in .NET 9 Preview 7, these desktop MonoVM nuget runtime packages are no longer available.
Type of breaking change

[ ] Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[X] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code may require source changes to compile successfully.
[ ] Behavioral change: Existing binaries may behave differently at run time.

Reason for change
There is no official .NET scenario that aligns with these desktop MonoVM runtime nuget packages.
Recommended action
If your application publish relies on these packages, we recommend staying on .NET 8 LTS while the application migrates to the default desktop runtime nuget packages.  Each of these platforms has a corresponding runtime nuget package.
Feature area
Other (please put exact area in description textbox)
Affected APIs
No response

there are so many team embed mono with game engine.
I hope you keep the original plan. upload the mono desktop runtime package.
More people and teams will use and verify it

@srxqds can we schedule some time to talk?  I would love to hear details about how these packages are used and if our corresponding Microsoft.NETCore.App.Runtime.* would be a viable option.
To connect, here is my linkedin profile: https://www.linkedin.com/in/jeff-schwartz-4ba91023/

hi，@jeffschwMSFT, I am so sorry I have no linkedin account, I can describe here as detail as I can.
we have emebed mono in UnrealEngine, we use c# as game script develop, We have been using it for many years.
And according to my understanding, many teams actually use this solution.

A few follow-up questions.
For some platforms is monovm built and package (eg. consoles, etc.) locally?  This breaking change is does not change the lifecycle of monovm.
For scenarios where these packages are used, have the Microsoft.NETCore.App.Runtime.* been explored?  What gaps exist to move to these?
.NET 8 will continue to have support for these packages which aligns with our lifecycle: https://learn.microsoft.com/en-us/lifecycle/products/microsoft-net-and-net-core.  So in the near term .NET 8 could continue to be a supported scenario.  We would be interested in hearing what a change to our default packages would look like.

ok, but when you not shipping nuget packages, we are starting to worry that you may be no longer supported
and close test coverage for mono desktop runtime.

I understand the concern.  This breaking change does not impact monovm's support for its primary shipping scenarios - iOS, Android, WASM, etc.  Internally we use versions of these builds for validation, but we will no longer ship them for the reasons noted above.
I hope that clarifies what is changing.

ok, than you for reply.


Upgrade Scorpio to 2.1.20

Description
Supersedes #120 pending task.

We'll be switching to Orion-LD.


Dialog updates

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release 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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 260111 run on x86_64-linux 1

1 package failed to build:

icesl

38 packages built:

certbot (python310Packages.certbot)
certbot-full
certbot.dist (python310Packages.certbot.dist)
debian-goodies
dialog
gammu
kabeljau
protonvpn-cli
protonvpn-cli.dist
protonvpn-cli_2
protonvpn-cli_2.dist
python310Packages.certbot-dns-cloudflare
python310Packages.certbot-dns-cloudflare.dist
python310Packages.certbot-dns-google
python310Packages.certbot-dns-google.dist
python310Packages.certbot-dns-inwx
python310Packages.certbot-dns-inwx.dist
python310Packages.certbot-dns-rfc2136
python310Packages.certbot-dns-rfc2136.dist
python310Packages.certbot-dns-route53
python310Packages.certbot-dns-route53.dist
python310Packages.python-gammu
python310Packages.python-gammu.dist
python311Packages.certbot
python311Packages.certbot-dns-cloudflare
python311Packages.certbot-dns-cloudflare.dist
python311Packages.certbot-dns-google
python311Packages.certbot-dns-google.dist
python311Packages.certbot-dns-inwx
python311Packages.certbot-dns-inwx.dist
python311Packages.certbot-dns-rfc2136
python311Packages.certbot-dns-rfc2136.dist
python311Packages.certbot-dns-route53
python311Packages.certbot-dns-route53.dist
python311Packages.certbot.dist
python311Packages.python-gammu
python311Packages.python-gammu.dist
wifish

@ofborg build icesl

Icesl is unfree.
Also it is broken :/


Improves formatting used on changelogs

Description

Describe your changes in detail
Why is this change required? What problem does it solve?
If it fixes an open issue, please put a link to the issue here.

Self-explained
Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] House cleaning
[ ] Bug fix
[ ] Enhancement
[ ] Breaking change
[ ] New feature
[ ] New release
[ ] Documentation

Additional details

Please, list here some additional details we should be aware of when reviewing your PR.

N/A

Codecov Report

Merging #157 (15ca061) into main (5adbf6c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #157   +/-   ##
=======================================
  Coverage   84.80%   84.80%
=======================================
  Files          11       11
  Lines         500      500
=======================================
  Hits          424      424
  Misses         76       76

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 5adbf6c...15ca061. Read the comment docs.


macOS support?

Minus the project file changes that would need to happen, would it be possible for the Nordic DFU library to target the macOS architecture?

It seems I've answered my own question. This library works on macOS with Cocoapods. Closing.


Migrating to react-testing-library

All of @Mensae's work from #2589 and #2473.

[ ] You have followed our contributing guidelines
[ ] Double-check your branch is based on dev and targets dev
[ ] Pull request has tests (we are going for 100% coverage!)
[ ] Code is well-commented, linted and follows project conventions
[ ] Documentation is updated (if necessary)
[ ] Internal code generators and templates are updated (if necessary)
[ ] Description explains the issue/use-case resolved and auto-closes related issues

Coverage decreased (-0.004%) to 99.642% when pulling 2eb3b94f9ec82c5a3a22c5acd75f4c00b9a044ca on react-testing-update into abf37d94dbad2f1dfac8e2621939adfa6f14699d on dev.

@Mensae Do you know what's happening here by any chance?

Weirdly this only happens in LTS...

🤔 intersting. I'll try to have a look at it today.

The good news: AppVeyor passes! Your fix is 👌!
The bad news: Travis has started failing us on Node 11.11 (not LTS) pretty much out of nowhere. It happens for all branches but we can't merge anything into dev until it's fixed. 😓

It's a bug between Jest and Node 11.11 (which is used by Travis to run our tests). I upgraded jest-cli and it should be fixed.

Yes! 🙂
Yes, I saw Travis failing but was scared to look right before bedtime. Hopefully the upgrade will fix it, fingers crossed

It's fixed. We just need someone else from the team to review before we can merge!

Excellent work @Mensae thank you, I've been wanting to move to react-testing-library at work but needed some frame of reference for it 🙌  This is a great addition, happy to merge. (and of course shoutout my buddy @julienben)


[Feature][Sort] Add MySQL connector on Flink v1.18

Description
Parent Issue #11420 [Feature][Sort] Support bounded MySQL source
Add MySQL connector for inlong-sort flink v1.18
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will get on with this issue in a week's time


[Storage] az storage container: Support --account-name for non-standard account URL

Related command
az storage container
Description

When use non-standard account URL, account name can't be parsed from URL, thus ValueError: Unable to determine account name for shared key credential is raised. In this PR, it will use credential dict if account_name is provided.
Close https://github.com/Azure/azure-cli/issues/23702.
Testing Guide
az storage account create --name <name> --resource-group <rg> --edge-zone microsoftrrdclab1 -l eastus2euap
az storage container create -n test --blob-endpoint <blob-endpoint> --account-name <name>

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Storage


Fix GlowingOverscrollIndicator bug

Description
I caught this bug during investigating #64149,
This is the improvement of #59015, @chunhtai @Piinks
Related Issues
None.
Tests
I added the following tests:
CustomScrollView overscroll indicator works if there is sliver before center and modify glow position
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

@chunhtai @Piinks Please review, thanks:)

@HansMuller Hi, Please review this change if it is convenient for you, thank you very much :)


[usb_dev, rv_dm, i2c] Updated waivers

Description
Given the stage that several blocks were for M2 we will need to prepare updated waivers for M2.5

[ ] USB_Dev updated waiver
[ ] RV_DM updated waiver
[ ] I2C updated waiver

For reference waivers general
I2C waiver, RV_DM waiver, USB V1/2 Waiver,

I think we can mark the I2C task as done because we will get I2C V2 sign-off done as tracked in this issue: https://github.com/lowRISC/opentitan/issues/18596 Thus we no longer need a waiver for V2.

I think we can mark the I2C task as done because we will get I2C V2 sign-off done as tracked in this issue: #18596 Thus we no longer need a waiver for V2.

Good point!
@moidx @GregAC let's discuss in the TL meeting tomorrow whether we actually need the V signoff waivers for M2.5.1 already.

Discussed in TL sync today - I2C will be closed without a waiver, hence marking that as done.

RV_DM waiver doc

The waivers have been updated for M2.5.1, hence this issue can be closed.


IE issue

Hello. Today I tried to use dropdown and found bug in ie. When I enabled search feature, search input is disabled(I couldn't input any data)

Could you create a plnkr where you replicate this issue so I can take a look at it?

Hello Pieter. I am so sorry/ I already found problem with ie. It was virus on my PC. I think we can close issue. Thank you very much for your time and support. The directive is very cool.

12:03 +03:00 от Pieter Kempenaers<EMAIL_ADDRESS>Could you create a plnkr where you replicate this issue so I can take a look at it?
—
You are receiving this because you authored the thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .

Cannot reproduce again


Backspace not working correctly for multi-paragraph content pasted from Google Docs

Jodit Version:  3.4.xxxxx
Browser: Chrome
OS:  Mac
Is React App:  False
Expected behavior:
Paste content containing multiple paragraphs from Google Docs into Jodit. Select 'Keep' in 'Paste as HTML' pop-up.
Place cursor in-between paragraphs and press backspace key.
Expected: blank line between paragraphs is removed
Actual: nothing happens
Place cursor at beginning of second paragraph and press backspace key.
Expected: blank line between paragraphs is removed
Actual: second paragraph is entirely removed
Actual behavior:
https://share.getcloudapp.com/OAuGXmxQ

After some investigation, I think at least part of the issue is that when pasting, the pasted content is created inside a div. Since the wrapTextNodes plugin is not recursive, it doesn't find and wrap any nested
 tags. Jodit then does not correctly recognize the
 tag when backspacing because it is not any of the expected formats

,

, or FEFF
.


Fix: dependencies compiled with incompatible versions of rustc

Crates with a rust-toolchain.toml override would forcibly install that toolchain and use it to compile these crates, leading to certain crates being compiled with different versions of rustc.
Setting the override with RUSTUP_TOOLCHAIN trumps the .toml override. Tested by deploying an axum hello-world app with dashmap, as well as a serenity app with the cache feature (which pulls in dashmap).

FYI I still experienced this issue with axum-macros and had to manually patch to a fork without rust-toolchain using [patch.crates-io] (i.e. this commit works (w.r.t this issue), but the previous commit doesn't).

Hey! This fix hasn't been deployed to production yet, so you'll need the work-around for a little bit longer. By the way, looking at your cargo.toml comments, we do support path dependencies since 0.8.0.

Ah gotcha, that makes sense.
It might be workspace dependencies, not path dependencies, that aren't working for me then 🤔 uncommenting the { workspace = true } dependencies gives:
2023-01-01T20:40:19.707969953Z ERROR {error="Build error: failed to parse manifest at `/opt/shuttle/shuttle-builds/recipe-shuttle/Cargo.toml`"} shuttle_deployer::deployment::queue: service build encountered an error

if I change all of those { workspace = true } to { path = "../whatever" } I see:
2023-01-02T13:51:44.004967421Z ERROR {error="Build error: failed to get `recipe-db` as a dependency of package `recipe-shuttle-fd2affd4-0a12-43ae-9a8b-c8d5875887a3 v0.1.0 (/opt/shuttle/shuttle-builds/recipe-shuttle)`"} shuttle_deployer::deployment::queue: service build encountered an error

I suspect this should be in a separate issue though - lmk if these are unexpected issues and I'll create one 😄

Hmm, it should work. :thinking: What is your cargo shuttle --version?

❯ cargo shuttle --version
cargo-shuttle 0.8.1

🤔

For the workspace dependencies, we aren't currently archiving the entire workspace but only the single crate. This is why the paths don't work and why the workspace toml can't be found (parsed). I've created a new issue for this #564


Update README.md

Updates
I updated README.md - please review and accept.

I probably shouldn't but ok 👍


Update subgraph example docs

Signed-off-by: Maksim Dimitrov<EMAIL_ADDRESS>Description:
Updates the subgraph example readme
Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Base: 70.50% // Head: 70.50% // No change to project coverage :thumbsup:

Coverage data is based on head (762903d) compared to base (3dce956).
Patch has no changes to coverable lines.

:exclamation: Current head 762903d differs from pull request most recent head 7d01728. Consider uploading reports for the commit 7d01728 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #607   +/-   ##
=======================================
  Coverage   70.50%   70.50%
=======================================
  Files          16       16
  Lines        1129     1129
  Branches      192      192
=======================================
  Hits          796      796
  Misses        282      282
  Partials       51       51

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[DDW-502] Bump cardano version to sync-stalling debug branch

This PR bumps Cardano version to revision which contains sync-stalling debug code by @karknu.

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@karknu will your branch be merged into Cardano release/2.0.0 branch?

@nikolaglumac No, I think that it is still a workaround and not a fix.

@karknu may I kindly ask you to merge latest Cardano release/2.0.0 branch into your branch? This will enable us to use EKG server... Thanks!

I canceled buildkite as this branch does not have a PR on cardano so hydra isn't building it. We're only testing windows builds at this time.

Just for the reference, here's the latest valid OSX/Linux build: https://buildkite.com/input-output-hk/daedalus/builds/3611


Refactor pilot test organization

Prior to this change we had a package per test, making an unneccesary
number of istio install/teardowns. Any test without a special set up can
just be in the top level pilot package. This required some changes to
the existing test in the top level (for some reason it was blocking the
Close() method when it wasn't using the "new style" of tests) and some
improved error reporting for trafficshifting.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

/test all

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-presubmit-tests-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test integ-istioctl-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-presubmit-tests-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test integ-istioctl-k8s-presubmit-tests-master

istio-lint-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test istio-lint-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-presubmit-tests-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test integ-istioctl-k8s-presubmit-tests-master

istio-lint-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test istio-lint-master

integ-mixer-k8s-presubmit-tests-master
2bb6b243639835ec9302a53e3b7a52f88955c372
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test e2e-simpleTests-distroless-master

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
310fa2e8a3d5bd93b622122f53683996250639ac
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
412326ed41727c3571ad226f08196f400f41201f
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
/assign @nmittler

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/pilot/OWNERS [howardjohn,nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
412326ed41727c3571ad226f08196f400f41201f
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
412326ed41727c3571ad226f08196f400f41201f
link
/test integ-mixer-k8s-presubmit-tests-master

istio-racetest-master
412326ed41727c3571ad226f08196f400f41201f
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
412326ed41727c3571ad226f08196f400f41201f
link
/test integ-mixer-k8s-presubmit-tests-master

istio-racetest-master
412326ed41727c3571ad226f08196f400f41201f
link
/test istio-racetest-master

integ-security-k8s-presubmit-tests-master
412326ed41727c3571ad226f08196f400f41201f
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
412326ed41727c3571ad226f08196f400f41201f
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
412326ed41727c3571ad226f08196f400f41201f
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
412326ed41727c3571ad226f08196f400f41201f
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Do not add to local volume cache if the volume driver's scope is global

Problem
Updating the local volume cache for a volume with global scope creates inconsistency in the cache data across hosts. Due to this inconsistency, the docker volume create command fails to create the volume if it is deleted outside the host where it was created and tried to create on the host where it was created earlier. The dockerd decides that the volume is still present due to entry in the local cache which cannot be decided based on local cache alone as the scope of the volume is global.
Not adding a global volume to local cache avoids the inconsistency in cache data across hosts in a cluster if the volume driver is designed for global scope. Hence, the need to handle stale entries in the cache is avoided if the volume is deleted outside the host, where the volume was created.
Carried from problem #27129 with a better approach.

PR #27274


Merge remote-tracking branch 'upstream/stable' into merge_stable

Conflicts:
test/runnable/cppa.d
test/runnable/extra-files/cppb.cpp

Auto-merge toggled on


Remove Warning of Deletion from Footer

By default all content is destroyed within 30 days, but users may now opt to have their encrypted content stored forever(?). This means we should remove the warning "This post is temporary. All new content is destroyed within 30 days"
This issue is intended to be a good introductory issue for learning the Priv.ly Project's build system.

@smcgregor I cannot replicate this bug on my system. I think this issue has already been resolved. Please look into it and close the issue.

Perhaps I was on an older branch at the time. Sorry for the wild goose
chase. I'll confirm then close.

Confirmed via grep.