Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: scan lit templates for type errors #2341

Closed
wants to merge 46 commits into from

Conversation

daniellacosse
Copy link
Contributor

@daniellacosse daniellacosse commented Jan 22, 2025

It works, just need to fix tests

@daniellacosse daniellacosse force-pushed the daniellacosse/ts-lit-plugin branch from bbaa2aa to 410f877 Compare January 22, 2025 22:15
@github-actions github-actions bot added size/S and removed size/XS labels Jan 23, 2025
@daniellacosse daniellacosse changed the title feat: add ts awarness to lit templates feat: scan lit templates for errors Jan 23, 2025
@daniellacosse daniellacosse changed the title feat: scan lit templates for errors feat: scan lit templates for type errors Jan 23, 2025
jyyi1 and others added 22 commits February 20, 2025 12:39
* revert to only the package.json changes

* minimize the number of package-lock changes

* webpack works!

* partial cleanup, need to regen package-lock still

* lock cleanup

* use 3.2.1

* add github issue
* feat(manager): move and update metrics cards in the server view

* getting close. trying to figure out this weird data model

* style update

* don't use reduce lol

* okay  it roughly works end to end

* add metrics messages

* access key tab

* update to use new "endpoint" - feedback next

* feedback

* endpoint implemented

* Update server_manager/www/app.ts

Co-authored-by: Vinicius Fortuna <fortuna@users.noreply.github.com>

* Update server_manager/model/server.ts

Co-authored-by: Vinicius Fortuna <fortuna@users.noreply.github.com>

* partial feedback

* feedback - needs test

* move implementatino

* fix

* fix unit formatting

* refactor for testing

* Update server_manager/www/shadowbox_server.ts

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* partial feedback

* simply method and remove from model

* make private

---------

Co-authored-by: Vinicius Fortuna <fortuna@users.noreply.github.com>
Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>
* feat(manager): resize window and left-adjust tabs

* fix sidebar

* fix spelling
* chore(manager): add translated strings for the new server tab

* Update server_manager/messages/master_messages.json

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* better descriptions

---------

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>
* fix(client/ios): update minimum deployment target to `15.5`

* Fix iOS Swift reference: `15.5` doesn't have a type property.
…2330)

This PR introduces a `callback` mechanism that allows TypeScript code to subscribe to events triggered in the Go code.

To demonstrate the functionality, this PR implements a VPN connection state change event. The TypeScript code can now subscribe to this event and receive updates on the VPN connection state.

This mechanism can be leveraged for both electron and the Cordova code.
* fix(manager): merge privacy statement message

* fix link
* feat(manager): add privacy statement and advanced metrics link

* Update outline-server-view.ts
* feat(manager): use server metrics row

* ok need to test and iterate

* unit cleanup

* everything is in, style cleanup

* fix old view

* adjust fonts and colors

* formatting cleanup

* address feedback

* oops forgot other ASes

* remove devices

* fix issues

* feedback

* Update app-root.ts
* feat(manager): data table primitive

* wip (switching branches)

* trying to figure out how to do the gaps with the css table

* time for example

* closer

* better than ever baby

* control component and cleanup left

* let's get it into review I guess

* missing license headers

* style fixes

* make data table generic

* improve sorting

* address feedback - check to make sure it works

* style adjustments

* event forwarding (?)

* streamline event forwarding
jyyi1 and others added 22 commits February 20, 2025 12:40
…ct (#2364)

This change ensures that both the TUN device and all associated NetworkManager connections are deleted.
This PR adds a 10-second timeout to `CloseVPN()`.  The `CloseVPN()` method can hang a very long time if there's outgoing traffic and the network is unreachable, due to lwIP holding a global lock.

#### Limitation

This is a workaround, so the user can quit the app. However, the lwIP lock *still* persist, requiring an **application restart**.
This PR enables WebSocket keys for the Outline Client on Windows.

- Removes the proxy IP exclusion logic from the routing service, preventing traffic leaks.
- The Go backend socket is now bound to the network interface connected to the internet.
…2370)

This PR signs two dependent binaries included in the NSIS installer. These binaries are not tracked by Electron's build process (they are included by the NSIS builder instead). So we need to properly sign them by ourselves.
* fix(client): validate the access key when the attr changes

* oops

* works in storybook, time to test on device

* re-namery

* empty vs. invalid
…gning (#2381)

This PR checks in signed pre-built Windows binaries directly into the repository and reverts #2370 . It guarantees that the binaries in the repository are authentic and unmodified.
* wip

* "integrated"

* working on formatting now

* formatting

* now to test integration

* okay i'm just futzing at this point

* revert

* make bandwidth responsive

* fix tests

* almost done, just gotta fix tests

* fixed

* feedback

* anticipate new shape
PiperOrigin-RevId: 726105634
* feat(manager): new access key data table integration

* can read...

* spinner

* pause for now

* tooltip fix

* revert FF

* screw it, delete stuff

* a little better

* colums is there

* handle nullables

* well, it works

* closer to desired styling

* screenshots and hoisting unsafeHTML

* pre-emptively merge the bandwidth row to deal with conflict ahead of time

* wip - fixing...

* put this back

* data is coming through

* gracefully degrade on old endpoints

* split timestamps

* icons fixed

* fix regressions

* subcard centering

* reference issue 2384

* computed properties

* fix tests

* last thing - the lag!

* okay. time for the final test

* done

* update stories

* tweaks

* faarsi

* format number

* address design review

* remove lastConnected column

* Update server_manager/www/views/server_view/server_metrics_row/bandwidth.ts

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* Update server_manager/www/views/server_view/access_key_data_table/data_table/index.ts

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* Update server_manager/www/views/server_view/access_key_data_table/data_table/index.ts

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* Update server_manager/www/views/server_view/access_key_data_table/data_table/index.ts

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>

* feedback

* fixes, works now

---------

Co-authored-by: Sander Bruens <sbruens@users.noreply.github.com>
PiperOrigin-RevId: 727059627
PiperOrigin-RevId: 727061287
* fix(client): better data limit bounds

* handle less than 1%
* fix(client): electron app doesn't respect reflected properties

* Update bandwidth.ts

* Update index.ts
…2403)

* feat(manager): create update notification bar component

* oops, link color

* tweaks
* fix(client): macos client now properly renders in dev mode, no need for warning

* remove chalk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants