nucleuscloud / neosync Goto Github PK

Reading neosync docs at night is blinding and we should support dark mode for them.

_{From SyncLinear.com | NEOS-447}

right now if it's a team account we route to members. this is kinda weird and we should just always route to the temporal page for consistency.

_{From SyncLinear.com | NEOS-499}

_{From SyncLinear.com | NEOS-435}

This happened when alisha switched the database out.

the frontend was trying to update the session storage to include the latest account but it kept being overwritten to the old one.

This is doubled by the fact that SWR keps trying to reach out to the jobs api and was met with a 403. It was doing it endlessly though. Why was it not backing off? Need to look into this.

_{From SyncLinear.com | NEOS-457}

_{From SyncLinear.com | NEOS-472}

We want to reduce our dependency on auth0 in favor or any OIDC compliant provider.

To do so, we need to refactor this table to be more generic.

Perhaps the new table should follow this schema: provider_name, provider_identifier

The auth0_provider_id column would be replaced by the following:

provider_name | provider_identifier
----------------------------------
        auth0 | google|[email protected]
        github| nickzelei
        google| [email protected]

_{From SyncLinear.com | NEOS-507}

_{From SyncLinear.com | NEOS-477}

weird that it has a started at value but nothing for the status?

_{From SyncLinear.com | NEOS-440}

It's probably good we support more than one auth provider to keep ourselves honest.

Today we only support Auth0 in neosync. We should also support Google

_{From SyncLinear.com | NEOS-496}

_{From SyncLinear.com | NEOS-438}

We need an interceptor that logs start and requests but that is built for connect.

Should be able to adapt it from this: https://github.com/grpc-ecosystem/go-grpc-middleware/tree/main/interceptors/logging

_{From SyncLinear.com | NEOS-462}

_{From SyncLinear.com | NEOS-425}

This is already pretty minimal thanks to next-auth.

However, we load in specific Auth0 named env vars today.

We should make this more generic and just ask for one set.

Next-auth technically supports multiple providers, but usually an OIDC provider is used as the front for those.

_{From SyncLinear.com | NEOS-508}

We could use some docs for installing neosync cli.

we have some pretty good ones over at the old nucleuscloud cli docs, we could use that as a starting template in terms of layout and what content we should expect.

https://docs.nucleuscloud.com/home/getting-started/installing-the-nucleus-cli

_{From SyncLinear.com | NEOS-449}

We do this now for go but need to do the same thing for npm.

we could also add a top level makefile to make this easier

we could also add all of this stuff to the dev dockerifles to make it more seamless.

_{From SyncLinear.com | NEOS-461}

Logo seems to scale to the size of the window.

It shouldn't do that 😃

_{From SyncLinear.com | NEOS-509}

Gives the ability to logout. Should clean up access/refresh tokens on disk if they are present.

This should also request a logout url from the server and open a web browser to log the user out of their session.

Check out similar logic in the nucleuscloud cli: https://github.com/nucleuscloud/cli/blob/main/cmd/logout.go

_{From SyncLinear.com | NEOS-503}

Might as well make all of the resources name clickable. most other tables have a uuid col and that is clickable.

_{From SyncLinear.com | NEOS-469}

Is your feature request related to a problem?

docs refer to baremetal and would like this.

Describe the solution you'd like

I like to use Fly.io, which uses a single dockerFile.

https://fly.io/docs/app-guides/multiple-processes/

So what we do is as part of the Docker boiler, you stuff in your binaries, which in neosync 2 or 3 I think.
Fly has the concept of Apps, and it will boost each App ( which is a binary ).

The other way that we can use Overwind, to run the binaries outside docker or inside docker.

https://github.com/DarthSim/overmind

Describe alternatives you've considered

Additional context

https://fly.io/docs/hands-on/install-flyctl/ is golang.

Fly can scale to zero and also scale per region
https://fly.io/docs/apps/scale-count/#:~:text=Scale%20an%20app%27s%20regions,its%20changes%20across%20these%20regions.

Thank you for your feature request – we love each and every one!

We saw that when we mounted the postgres db volume for neosync-db there were missing files and missing permissions.

The problems went away when removing the passthrough volume mount.

Why is temporal not suffering from this?

_{From SyncLinear.com | NEOS-458}

_{From SyncLinear.com | NEOS-454}

The input box and copy button should only show when we have the api key contents

This is the /<team>/settings/api-keys/:id page

_{From SyncLinear.com | NEOS-483}

_{From SyncLinear.com | NEOS-445}

_{From SyncLinear.com | NEOS-470}

Similar to the metadata we build into the CLI, we should do the same for the backend.

It would be great to then expose this in the App or API somehow.

Should this be authenticated if auth=true?

_{From SyncLinear.com | NEOS-434}

The link is correct, but somehow you are re-routed back to the default account.

Must have something to do with this useeffect:

  useEffect(() => {
    if (isSystemAppConfigDataLoading || isAccountLoading) {
      return;
    }
    if (systemAppConfigData?.isAuthEnabled && account?.name) {
      return router.push(`/${account?.name}/settings/members`);
    } else {
      return router.push('/personal/settings/temporal');
    }
  }, [
    account?.id,
    isAccountLoading,
    systemAppConfigData?.isAuthEnabled,
    isSystemAppConfigDataLoading,
  ]);

_{From SyncLinear.com | NEOS-498}

I havent been able to reproduce this on localhost, just staging for some reason.

account switcher goes away when you click on the home button (but none of the other pages?)

auth redirect doesnt always happen

Clicking around enough eventually logs you in.

need to figure out what the deal is here

Home page seems to be totally broken

_{From SyncLinear.com | NEOS-497}

_{From SyncLinear.com | NEOS-381}

we mount volumes to the filesystem so it's imperative that this gets safelisted in mac docker desktop otherwise it doesnt work.

_{From SyncLinear.com | NEOS-459}

_NEOS-460

job status currently doesn't support enough states and map correctly to temporal.

1. Remove status field from Jobs db table.
2. Map Neosync status to temporal status (pauseStatus, scheduleStatus)
3. Update FE to support pause status and schedule status

_{From SyncLinear.com | NEOS-442}

_{From SyncLinear.com | NEOS-450}

_{From SyncLinear.com | NEOS-424}

Need to publish this package upon release

_{From SyncLinear.com | NEOS-479}

Need a way to set account id in CLI.

Either as context or as flag.

trigger, list, sync

_{From SyncLinear.com | NEOS-467}

_{From SyncLinear.com | NEOS-436}

We no longer use this and is a source of bugs as it is still present on the database model

_{From SyncLinear.com | NEOS-466}

The action Cancel on the list page as well as "Cancel Run' button on the run details page is always there regardless of the run status.

We should be smarter here and only show this if the run is active.

_{From SyncLinear.com | NEOS-465}

Both ordered and unordered lists do not have a bullet or numbers rendered in them.

We should fix this.

_{From SyncLinear.com | NEOS-451}

need to update margins and check styling

_{From SyncLinear.com | NEOS-453}

we shouldnt allow malformatted protos.

I thought buf lint did this but it doesn't seem to care if buf fmt returns changes.

_{From SyncLinear.com | NEOS-423}

_{From SyncLinear.com | NEOS-431}

right now the random float transformer returns 17-18 digits for every number which might be too much precision for some use-cases.

someone should be able to specify a length param that the returned value should abide by

_{From SyncLinear.com | NEOS-491}

• strings, ints, floats, etc.
• handle nils for these values in the transformers, if the user doesn't pass anything in, we should just randomize between the min and max

_{From SyncLinear.com | NEOS-485}

_{From SyncLinear.com | NEOS-429}

this is to test the github<>linear integration

_{From SyncLinear.com | NEOS-416}

we could either render the home button as the absolute link, or persist the last known selected account. I kinda like the home button always being the base url, then letting the home nav take you to the jobs page of the last known selected account. that is more complex however.

_{From SyncLinear.com | NEOS-500}

_{From SyncLinear.com | NEOS-501}

write guides for:

• using the neosync github action and cli to sync CI databases
• creating a job
• configuring authentication

_{From SyncLinear.com | NEOS-433}

Both the app and api are pretty dependent on auth0. We should make this instead generic for oidc providers.

_{From SyncLinear.com | NEOS-441}

_{From SyncLinear.com | NEOS-430}