profile
viewpoint
Roberto Wesley Overdijk RWOverdijk SpoonX Studio Heerhugowaard https://medium.com/@r.w.overdijk

RWOverdijk/AssetManager 216

AssetManager written for zf2. Managing assets for zend framework 2

balderdashy/sails-mysql 190

MySQL adapter for Sails.js/Waterline

BlueHotDog/sails-migrations 157

The missing, migrations, arm of the octopus

rmevans9/react-native-router-flux 1

First Declarative React Native Router

RWOverdijk/AssetLoader 1

AssetLoader module for ZF2

RWOverdijk/2048-android 0

The android port of the 2048 game (for offline playing)

RWOverdijk/about 0

About SpoonX.

RWOverdijk/about-1 0

About SWAN stack

RWOverdijk/admin-on-rest 0

A frontend framework for building admin SPAs on top of REST services, using React and Material Design

issue commentFirebaseExtended/flutterfire

firebase_auth onAuthStateChanged not updated on linkWithCredential

I get that, I was talking about triggering the auth state change.

erf

comment created time in 2 days

issue commentFirebaseExtended/flutterfire

firebase_auth onAuthStateChanged not updated on linkWithCredential

I know this repo is not for the firebase JS web sdk, but I have the same issue there. I'm signed in as an anonymous user and upon linking with a credential it does not trigger the onAuthStateChanged.

Normally I wouldn't comment on an unrelated repo but I think these issues might be related somehow. .reload() also doesn't do anything for me so that also seems the same.

erf

comment created time in 3 days

issue commentfacebook/react-native

Rename TextInput prop "autoCompleteType" to "autoComplete"

Bump. What's the status of this, will it be aligned?

necolas

comment created time in 6 days

startedtholman/elevator.js

started time in 7 days

issue commentreact-native-community/discussions-and-proposals

Support for iOS 14 App Clip

This would be really cool. Not sure if this is something I should add here but android instant apps could be a nice addition as it provides something similar.

In terms of file size I'm thinking expo would also need a different approach. Again, not sure if this is useful to bring up here.

Finally, I read somewhere that it can be a separate app, but it needs to be in the same archive. Wouldn't that mean that a "RN lite" of sorts could do the trick, and then the only other thing needed is an approach to bundle the archive? These apps can be smaller because the features you're supposed to build with them are limited anyway.

My first thought would be utilities for basic clips with good instructions.

vrgimael

comment created time in 7 days

created tagMaaloptijd/desktop

tagv1.0.0

Desktop binaries for Maaloptijd

created time in 16 days

push eventMaaloptijd/desktop

Roberto Wesley Overdijk

commit sha 4236b62220b0472e915763c368b9edb9115b6efa

Update README.md

view details

push time in 16 days

create barnchMaaloptijd/dashboard-desktop

branch : master

created branch time in 16 days

created repositoryMaaloptijd/dashboard-desktop

Desktop binaries for Maaloptijd

created time in 16 days

issue commentbenwinding/react-admin-firebase

High Reads

I get that, and this would probably use virtualized lists of sorts to keep things snappy.

I agree with you, but I still think it should be a differently named component.

mzgnr

comment created time in 16 days

issue commentbenwinding/react-admin-firebase

High Reads

I like that. A user will almost never look for a specific page number, and will much sooner use search or filters. So in terms of pagination I'd say an infinite scroll (with one of those "click to load more" fallbacks) would do the trick.

Same features, less reads. I'd say it's pretty clever. It deviates from react-admin so I think it should work as a plugin of sorts though. Allow for the regular ra List to be used, or the InfinitList from ra-firebase to be used for infinite scrolling.

I like this! 😄

mzgnr

comment created time in 16 days

issue commentexpo-community/standard-version-expo

Android increments are wrong when using prereleases

I suppose the build numbers should just increment and the version should remain unaltered except for the first time it runs, where it should be bumped to the next version.

The challenge there is remembering that this happened I suppose, unless you can use the package.json version to verify.

byCedric

comment created time in 19 days

issue openedbenwinding/react-admin-firebase

How to use realtime

I found the watch option and upon trying to use it noticed that it doesn't update my list when I make changes in the firestore console. How should this feature be used?

created time in 21 days

issue commentbenwinding/react-admin-firebase

High Reads

@benwinding I think that's maybe not the best idea. You'll get a whole new world of pain if anyone changes a record and your cursors are suddenly wrong. But after trying and spending time on this I came to just one conclusion personally.

I have just accepted that firestore isn't ideal if you're going to have a lot of data. It's fine to get started, but quickly just becomes expensive and difficult to work with. I'm finally ready to stop trying to make it do things it just wasn't built to do. :p

mzgnr

comment created time in 21 days

issue commentakveo/eva-icons

Instagram Icon

There's a linkedIn icon, but no instagram. Confirmed: eva design is for old people.

All joking aside, can we help? There are plenty of existing icons out there so it shouldn't be too hard to add an icon, right?

nseb

comment created time in 22 days

issue openedmoment/moment-timezone

File size huge with node

I've googled around a bit and found suggestions to reduce the file size with webpack.

I'm using this lib with node and so I don't really use webpack. Installing this lib comes in at 6.3M (as it includes moment, too).

Any suggestions on how to reduce this? 89%(!) of our entire codebase is now moment(-timezone).

created time in a month

issue commentexpo-community/standard-version-expo

Android increments are wrong when using prereleases

o/ I just tried releasing a prerelease again which reminded me of this issue.

Any pointers on how I might help resolve this?

byCedric

comment created time in a month

issue commentnecolas/react-native-web

Image: new babel/webpack loader

I understand the reasoning, I get it, but I disagree. It sucks that I can't load images that contain an @ in the name. Now I renamed all images and I always have to load the correct one myself, even on mobile.

I think that at least having a clear guide on this would be much better than that.

necolas

comment created time in a month

issue openedKlemen1337/node-thermal-printer

Find printer in network

Hey!

I love this module and it works really well. Thank you for this.

Currently I am configuring the IP address of the printer manually, but I would like to detect it on the network automatically. Is there a way I can do this? I can use "auto" for connected printers, but does this also work for printers on the network? Because it doesn't seem like it. If there's a doc I didn't find/read I apologise. I did my best to find it before asking a question.

Thanks again, big fan 😄

created time in a month

issue commentgoogleapis/google-cloud-node

Unacceptable cold start get() performance

So, I've successfully moved my stuff to app engine to test and the initial cold boot is still around 4 seconds, but now this only happens once because I've set my min instances to 1, it's a lot faster now. Also because I've limited my max instances and tweaked my target_cpu_utilization and max_concurrent_requests it's a lot faster. It no longer (wrongly) thinks it needs to scale up.

So now my schedulers and triggers are deployed to firebase functions and my api has been deployed to app engine.

Again, not a solution, but for me at least a workaround I can live with.

mbleigh

comment created time in a month

issue commentgoogleapis/google-cloud-node

Unacceptable cold start get() performance

I don't know if this is useful to anyone, but I'm moving everything to app engine, this gives more control and might help with performance. It doesn't solve the actual issue, of course, but it might be a workaround to get the performance needed.

If this is a dumb/unhelpful comment let me know, I can delete it.

mbleigh

comment created time in a month

PullRequestEvent

PR closed fnando/i18n-js

allow options for localization and add spec
+37 -14

1 comment

3 changed files

RWOverdijk

pr closed time in a month

pull request commentfnando/i18n-js

allow options for localization and add spec

I'm not sure why Travis isn't triggered

RWOverdijk

comment created time in a month

PR opened fnando/i18n-js

allow options for localization and add spec
+37 -14

0 comment

3 changed files

pr created time in a month

push eventRWOverdijk/i18n-js

RWOverdijk

commit sha 9c458efc89b9b65a663342b6bd152f5500b0ba92

allow options for localization and add spec

view details

push time in a month

PR closed fnando/i18n-js

add spec for inline localization

Tried sticking to the code style (double quotes and odd comma placement and such). 😄

+37 -14

8 comments

3 changed files

RWOverdijk

pr closed time in a month

pull request commentfnando/i18n-js

add spec for inline localization

Anyway, done.

RWOverdijk

comment created time in a month

pull request commentfnando/i18n-js

add spec for inline localization

GitHub can do that nowadays I think?

RWOverdijk

comment created time in a month

push eventRWOverdijk/i18n-js

RWOverdijk

commit sha 5b478e9aa92000ab5fb239d1d7a90a3980842659

refix indentation

view details

push time in a month

issue commentreact-native-community/react-native-webview

Apple pay in webview

@YousefAlsbaihi javaScriptEnabled*

And Without any javascript the webview becomes pretty useless, which is why stripe doesn't work (it needs JavaScript).

MaksymLebedevs

comment created time in a month

push eventRWOverdijk/i18n-js

RWOverdijk

commit sha 90f82fb237849801a3691453d8723820d01c2532

refix indentation

view details

push time in a month

Pull request review commentfnando/i18n-js

add spec for inline localization

      switch (scope) {       case "currency":-        return this.toCurrency(value);+        return this.toCurrency(value, options);       case "number":-        scope = this.lookup("number.format");+        scope = this.lookup("number.format", options);         return this.toNumber(value, scope);-      case "percentage":-        return this.toPercentage(value);+        case "percentage":

@PikachuEXE Because I made a mistake 😄

RWOverdijk

comment created time in a month

push eventRWOverdijk/i18n-js

RWOverdijk

commit sha 51ab3adcaa469be76199fa5808d6084b27092238

pass options on to nested methods

view details

push time in a month

pull request commentfnando/i18n-js

add spec for inline localization

@PikachuEXE I gave it a go and update the PR. Basically I just pass on the options to the lookup.

RWOverdijk

comment created time in a month

issue commentfnando/i18n-js

Date format for specific locale

Maybe it'd be nicer to be able to get a new instance that uses a single locale? Here's how I solved something like that now:

const i18n = require('i18n-js');
const CURRENCY_FORMAT = { unit: '$', precision: 2, format: '%u%n', sign_first: true, delimiter: ',', separator: '.' };

export class I18n {
  private readonly locale: string;

  constructor(locale: string = 'en') {
    this.locale = locale;

    I18n.loadLocale(locale);
  }

  public static t(locale: string, key: string, params?: { [key: string]: string }): string {
    return (new I18n(locale)).t(key, params);
  }

  public static loadLocale(locale: string): typeof I18n {
    i18n.translations = i18n.translations || {};

    if (!i18n?.translations[locale]) {
      i18n.translations[locale] = require(`./locales/${locale}`).default;
    }

    return this;
  }

  public getLocale () {
    return this.locale;
  }

  public t(key: string, params?: { [key: string]: string }): string {
    return i18n.t(key, { ...params, locale: this.locale });
  }

  public toCurrency (cents: number, options?: any) {
    return i18n.toNumber(cents / 100, i18n.prepareOptions(
      options,
      this.t('number.currency.format'),
      this.t('number.format'),
      CURRENCY_FORMAT,
    ));
  }
}

This just always adds the locale when calling translate, as a scoped solution. I had to reimplement toCurrency as you can see, but otherwise I would like to (from the docs):

// accepted formats
I18n.l("date.formats.short", "2009-09-18", { locale: someLocale });           // yyyy-mm-dd
I18n.l("time.formats.short", "2009-09-18 23:12:43", { locale: someLocale });  // yyyy-mm-dd hh:mm:ss
I18n.l("time.formats.short", "2009-11-09T18:10:34", { locale: someLocale });  // JSON format with local Timezone (part of ISO-8601)
I18n.l("time.formats.short", "2009-11-09T18:10:34Z", { locale: someLocale }); // JSON format in UTC (part of ISO-8601)
I18n.l("date.formats.short", 1251862029000, { locale: someLocale });          // Epoch time
I18n.l("date.formats.short", "09/18/2009", { locale: someLocale });           // mm/dd/yyyy
I18n.l("date.formats.short", (new Date()), { locale: someLocale });           // Date object

In any case, I did make a PR with the specs: https://github.com/fnando/i18n-js/pull/569

RWOverdijk

comment created time in a month

PR opened fnando/i18n-js

add spec for inline localization

Tried sticking to the code style (double quotes and odd comma placement and such). 😄

+24 -1

0 comment

2 changed files

pr created time in a month

push eventRWOverdijk/i18n-js

RWOverdijk

commit sha 5d871bfa42a15393096a4fef4113adef4be10e7e

add spec for inline localization

view details

push time in a month

fork RWOverdijk/i18n-js

It's a small library to provide the I18n translations on the Javascript. It comes with Rails support.

fork in a month

issue openedfnando/i18n-js

Date format for specific locale

Maybe I don't understand this lib very well, but it seems like there's only a global locale.

What I want to do is call translate and localize using a locale. This already works for translate, I can just say:

i18n.t('some.string', { locale: 'nl' })

But I can't find the equivalent for localize. I'm using this library server side and so using a global locale will cause conflicts unless I set the locale before every call to localize.

Is there something I can do to force a locale for these calls? I tried using { locale: 'nl' } as the second argument but it's not working.

created time in a month

startedmanojVivek/responsively-app

started time in a month

pull request commentauth0/node-jsonwebtoken

Added API endpoint for token refresh

You're commenting on an issue from 2016 dude.

But to answer your question: every form of session/auth has the same issue. Cookies: get the token. JWT: get the token. oauth: get the token.

Just make it really really difficult to steal the token. 😄

jppellerin

comment created time in a month

issue commentreact-native-community/react-native-webview

Apple pay in webview

@tdoran I did get it working. And I'll say this once and never repeat again (so if I link anyone to this comment in the future: I'm sorry but I'm not sorry).

I get that it's tempting to want to communicate between the app and the iframe/webview/browser screen... But don't. It's a dumb idea. It's like putting in a screw using a cement mixer. You'll get it to work eventually, maybe, but you'll also have cement everywhere and a bunch of people trying to lock you up for being insane.

And I get to say that, because I tried and I was also dumb for trying. These browser screens simply aren't made to do this work. And you'll safe yourself a lot of headaches if you simply don't even try.

Instead, make it not matter. You çan deep link back to the app when you're done, you can use websockets to listen for events on a common source, you can set up snapshot listeners with firestore (if that's your thing), you can even use polling after the browser window closes itself or the app state changes (because remember! The user can exit the app and open safari). There are a lot of possibilities, but communicating between the two without viewing them as entirely separate applications is not going to work in a stable manner.

So, I love you, all of you, silly kids. But don't make the same mistake I made. Un-stuck your mind from the "how to do this" mindset, and embrace the "lol, should I even be trying this?" mindset.

MaksymLebedevs

comment created time in a month

pull request commentauth0/node-jsonwebtoken

Added API endpoint for token refresh

Then you call the police. Theft is not okay.

image

jppellerin

comment created time in a month

issue commentTakeawayAPI/node-takeaway

Add order placement

Aah I misunderstood, this API is from the consumer side, not the restaurants. Still interesting though. I might take a look later and see what I can do 😄

DanielHuisman

comment created time in a month

issue commentbenwinding/react-admin-firebase

High Reads

@adamf92 Nothing yet for paging. The more I learn about firebase/firestore the more I regret choosing to use it 😅

mzgnr

comment created time in a month

issue commentkozmoz/thuisbezorgd-scraper

Status change functionality

@moparlakci Thanks! :) Do you have good experiences with this integration?

moparlakci

comment created time in a month

issue commentTakeawayAPI/node-takeaway

Add order placement

This library might be exactly what I've been looking for. Are you still working on it? Any word from takeaway if they're happy with this? 😄

DanielHuisman

comment created time in a month

issue commentkozmoz/thuisbezorgd-scraper

Status change functionality

@moparlakci Where can I find info on this API? I'd be very grateful if you could share that!

moparlakci

comment created time in a month

issue commentbenwinding/react-admin-firebase

High Reads

I can see your point. It is an unfortunate model to use with numbered pagination. Even using offset() in a function won't solve anything when it comes to reads...

Can't you just store the cursor value in redux? There's always a last id and as long as there's a sorting set up your results will be predictable. Then there's no need to store a cursor in local storage (which makes it unsharable) or the database (which breaks multi-screen features).

I mean sure, fetch the first 223 pages if needed. It's still less than fetching everything and it's unlikely someone would jump straight to page 223. One such example is orders. Who'd go back in time several months to find an old order? Some might, but then it's still a lot less reads.

I'd be interested in hearing more details about how this works @RWOverdijk

The fuzzy search? We just have on create, delete and update triggers that keep the "name" in a large array, which we use in the browser for fuzzy search. It won't work for everything but works fine for out specific case.

mzgnr

comment created time in a month

issue commentfirebase/firebase-admin-node

UserRecord is missing isAnonymous

Why is this closed? There's no proper reply here.

Inconsistent for sure.

eskan

comment created time in 2 months

issue commentakveo/react-native-ui-kitten

JS file support for custom mappings with metro

@artyorsh I understand. I personally wouldn't mind it doing that on changes to the main file only initially, I guess it's a trade-off. Or maybe it should only be used for production builds, and dev just doesn't (if provided mapping isn't JSON). I know it's not as pretty, but at least it keeps working for dev and remains optimised for builds?

RWOverdijk

comment created time in 2 months

issue commentbenwinding/react-admin-firebase

High Reads

It's a FieldValue increment and decrement. That part is really simple, it's the pagination itself that'll be a challenge. 😅

AS to the fuzzy search it's for out inventory system (we're a marketplace of sorts) and each venue gets a cache doc that we lazy load.

mzgnr

comment created time in 2 months

issue commentbenwinding/react-admin-firebase

High Reads

We have now created a --stats-- collection which holds metadata for all the other collections, such as the document count. All this really required were two hooks (create, delete) to maintain the counter.

Using this we can now start paginating the results. As to fuzzy search we've also made our own solution, which is a large document that contains everything that is searchable. We're well below 2kb and the limit of a doc is 1mb, so this is perfect. Also means there's only 1 read for fuzzy search where it's needed.

@benwinding do you have pointers as to how we could implement the pagination part on top/inside of this library?

mzgnr

comment created time in 2 months

issue commentfacebook/react-native

Add support for AlarmManager in Timing to allow proper handling of long timers

Because it should be fixed in the offending libraries. And the offending libraries don't see a problem with the way it has been implemented.

So nobody will fix it because both sides don't think it's broken.

They can be shamed a little bit for this 😄

astreet

comment created time in 2 months

issue openedakveo/react-native-ui-kitten

JS file support for custom mappings with metro

Based on this comment

🚀 Feature Proposal

It's already possible to use JS based mappings. It's what we're currently doing to keep things organized (without it UI Kitten's system would be far less appealing).

However, using it with metro for performance boost doesn't work, it requires JSON.

Motivation

Having one massive JSON file is not really an option. The reason myself (and team) enjoy working with the eva system so much is because of the way we can bundle mappings with our components. Our custom mapping file more or less looks like this (left most out to keep it short):

import QuantityStepper from '../components/QuantityStepper/mapping'
import ExpansionPanel from '../components/ExpansionPanel/mapping'

export default {
  components: {
    QuantityStepper,
    ExpansionPanel,
  }
}

This means our mappings are nice and only as tall as they need to be.

Example mapping.js:

export default {
  meta: {
    scope: 'all',
    parameters: {},
    variantGroups: {},
    states: {},
    appearances: {
      default: {
        default: true
      }
    }
  },
  appearances: {
    default: {
      mapping: {}
    }
  }
}

Example

It already works with js-based mappings, only the metro config doesn't like it yet.

created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

@artyorsh It does look like customMappingPath needs to be json for metro, and I'm using js everywhere. But maybe that changed for v5, I'm still on v4.

andersravn

comment created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

Oh heck yes, I did not know that. 😄

andersravn

comment created time in 2 months

startedmicrosoft/react-native-macos

started time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

I started on expo web, but quickly found the bundle size to be abnormally large. Our website is almost 4MB. On top of that prerendering (which is what you'll need for unfurling and OG reading) doesn't work properly (and requires a lot of work to get decent).

Next is the right way I think.

In regards of my approach, it's still not done but I am looking into exported style objects (just POJO) and then a template file (much like .web.js and .ios.js) for web and mobile. They both use the styles their own way (StyleSheet.create and style modules in next). It does mean I have some work to do in terms of a style system for web, but that's the cost so far...

andersravn

comment created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

My experience with all of it is that it feeks forced and hacky and I am looking into different approaches altogether.

I want to share components between platforms but I don't want that to come with a massive burden (think RN web and UI kitten). So I'm working on a way to share the logic and even styling between the two platforms but have the structural logic be separate (View vs div for example).

I think I'm on to something here but... Who knows.

andersravn

comment created time in 2 months

issue commentbenwinding/react-admin-firebase

High Reads

A crazy idea would be to use something like WatermelonDB to keep a local copy... We'd get all the features needed and it will feel fast.

mzgnr

comment created time in 2 months

issue commentbenwinding/react-admin-firebase

High Reads

This. We have a couple thousand records in multiple collections now and it's becoming expensive real quick.

In terms of "the number of records" a meta field could be made that keeps track of this. It's a common approach using FieldValue.increment. As with the other (recent) additions to this library I think it should be optional as it adds logic to your data that otherwise wouldn't exist. This is a very important feature though as otherwise this causes a dashboard to have a TTL.

Regarding fuzzy-search, this would be a trade-off that can easily be re-enabled using something like algolia if really needed. We're almost never using fuzzy search for example, so it wouldn't be a huge deal.

Maybe this can be a group effort? We can invest some time in this, anyone up to help?

mzgnr

comment created time in 2 months

issue closedwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

Get this error when playing a notification sound:

play() can only be initiated by a user gesture

Is this for all sounds?

Also I'm curious how to clean up sounds when I'm done with them, they seem to linger in the DOM.

closed time in 2 months

RWOverdijk

issue commentwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

Huh. Howler does work.

RWOverdijk

comment created time in 2 months

IssuesEvent

issue commentwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

Correction, even after interacting with the page I get this message.

RWOverdijk

comment created time in 2 months

issue closedwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

Get this error when playing a notification sound:

play() can only be initiated by a user gesture

Is this for all sounds?

Also I'm curious how to clean up sounds when I'm done with them, they seem to linger in the DOM.

closed time in 2 months

RWOverdijk

issue commentwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

@wle8300 MacOS. :)

The issue is only if the user hasn't interacted with the page yet; which makes sense so it's not a huge deal.

Interested in creating a PR for this?

Sure thing, might take a bit though.

RWOverdijk

comment created time in 2 months

issue openedwle8300/uifx

play() can only be initiated by a user gesture and cleaning up

Get this error when playing a notification sound:

play() can only be initiated by a user gesture

Is this for all sounds?

Also I'm curious how to clean up sounds when I'm done with them, they seem to linger in the DOM.

created time in 2 months

issue commentexpo/expo-cli

Next: withExpo not working with all modules

I could try that but I stopped using expo. Want me to close this issue?

RWOverdijk

comment created time in 2 months

issue commentSpoonX/aurelia-authentication

setTimeout(ttl) is not persistent

Ouch, cognito. That sucks.

I hope someone can help you but it's not me.

fieldfoxWim

comment created time in 2 months

issue commentreact-native-community/react-native-webview

Apple pay in webview

I don't know to be honest. I'm also using expo and I couldn't get it to work here.

MaksymLebedevs

comment created time in 2 months

issue commentreact-native-community/react-native-webview

Apple pay in webview

@ramisalem That depends on your method of choice. Tipsi does this (when it works). But I'd just assume it works, for example with stripe the button won't even show up if it's not supported.

About the jokes, we were just laughing at typos which in hindsight was a bit rude. Sorry for that.

MaksymLebedevs

comment created time in 2 months

pull request commentRWOverdijk/AssetManager

WIP: Migration to Assetic framework

Nice :) Could we get some more eyes on this?

thomasvargiu

comment created time in 2 months

issue commentreact-native-community/react-native-webview

Apple pay in webview

Oh look, you do have time! :trollface: Maybe you can fix some issues? 😇

MaksymLebedevs

comment created time in 2 months

issue commentreact-navigation/react-navigation

Direct link not building hierarchy as desired

Web fundamentally doesn't work that way

I get why you'd say that. And you're absolutely right when talking about the browser's back button. I agree with you there. It would be weird to push pages on to the history. The UX just needs to have a clear path back to previous screens.

That being said, that's the browser history, what about nav state? If there is a back button inside of the app itself for example (go back a screen, which is a very common use case). Shouldn't that go back? Again, not trying to be difficult 😅 I get not touching the browser history, but the nav state should be built up the way it was.

RWOverdijk

comment created time in 2 months

issue commentzeit/swr

Support socket request like sockjs-client or socket.io ?

@nandorojo Nope, that was once again exactly what I was looking for! Thanks :)

ycjcl868

comment created time in 2 months

issue commentIjzerenHein/firestorter

Nextjs SSR example

Looking into this, I think that perhaps firestorter or even mobx wouldn't make sense with next.js.

I found this article which explains how you could use mobx in next, but it feels clunky to me.

Zeit's swr seems capable of listeners and such as well. Working with small functional components with swr should yield the same result as working with firestorter and mobx.

By this I mean that you could set up a new onSnapshot listener on a collection/document and use swr's mutate to maintain it locally (so, use Mode.On basically).

To make it "smart" it'd have to be an effect with counters to check if anyone is observing though, so something could be done there.

But if all that is needed is a check to see if anyone is listening to the data I wouldn't use mobx for that. It's a pretty large library for something that could be solved with useEffect. The obvious downside is that you basically have to use hook. But that doesn't seem avoidable anymore anyway so... Why not.

I hope my ramblings help anyone. If not, well, at least I got to ramble.

sne11ius

comment created time in 2 months

issue commentzeit/swr

Support socket request like sockjs-client or socket.io ?

This is exactly what I was looking for. But won't this call the subscriber multiple times causing many onSnapshot listeners? On blur/focus for example?

ycjcl868

comment created time in 2 months

issue commentreact-navigation/react-navigation

Direct link not building hierarchy as desired

Thanks for the reply.

I think you should reconsider. I do believe this should also work with React Native, especially if you have nested routes.

Is this something that goes in a RFC?

RWOverdijk

comment created time in 2 months

issue commentreact-native-community/react-native-webview

Apple pay in webview

@ramisalem You wrote "massages" instead of "messages". It was a joke. But it didn't land.

MaksymLebedevs

comment created time in 2 months

pull request commentsoftware-mansion/react-native-gesture-handler

fix(web): enable text selection in GestureHandler

@xn3cr0nx React-navigation creates new releases about 293847 times a day, so chances are you just need to update again. I dropped react-navigation for web so I can't verify for you.

jaulz

comment created time in 2 months

issue commentIjzerenHein/firestorter

Nextjs SSR example

Did either of you happen to do this yet? I'm trying to get it working now and... Well, let's just say it's not.

firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error: firestorter: cannot get firestore: Error:....

etc.

sne11ius

comment created time in 2 months

issue commentreact-native-community/react-native-webview

Apple pay in webview

I like that your code gives massages

MaksymLebedevs

comment created time in 2 months

issue commentfirebase/firebase-js-sdk

Firebase Analytics doesn't work for Web app

check that your browser's adblock/privacy settings are not deliberately blocking it

This. I'm using Brave and it's not having it. So Google Analytics is essentially useless for our platform then?

agordeev

comment created time in 2 months

issue openedexpo/expo-cli

Next: withExpo not working with all modules

<!--

Required format for issues

Please make our job easier by filling this template out to completion. -->

Description

Using this guide I managed to setup expo + next love. It works great until you try to add something like ui-kitten + react native svg, then everything explodes and sadness splashes everywhere:

node_modules/react-native/Libraries/Utilities/warnOnce.js:15
const warnedKeys: {[string]: boolean} = {};
      ^^^^^^^^^^

SyntaxError: Missing initializer in const declaration

Expected Behavior

Less explosions, more love.

Observed Behavior

<!-- What actually happened when you performed the above actions?

If there's an error message, please paste the full terminal output and error message in this code block: -->

Oh no, I guess I ruined the format of the issue. 

node_modules/react-native/Libraries/Utilities/warnOnce.js:15
const warnedKeys: {[string]: boolean} = {};
      ^^^^^^^^^^

SyntaxError: Missing initializer in const declaration

Environment

  Expo CLI 3.20.1 environment info:
    System:
      OS: macOS 10.15.4
      Shell: 5.7.1 - /bin/zsh
    Binaries:
      Node: 12.13.0 - ~/.nvm/versions/node/v12.13.0/bin/node
      Yarn: 1.13.0 - /usr/local/bin/yarn
      npm: 6.12.0 - ~/.nvm/versions/node/v12.13.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    IDEs:
      Xcode: 11.4.1/11E503a - /usr/bin/xcodebuild
    npmPackages:
      expo: ^37.0.0 => 37.0.5
      react: ~16.9.0 => 16.9.0
      react-dom: 16.9.0 => 16.9.0
      react-native: ~0.61.4 => 0.61.5
      react-native-web: ^0.11.7 => 0.11.7
    npmGlobalPackages:
      expo-cli: 3.20.1

Reproducible Demo

Just expo + that guide + ui-kitten quickstart.

Workaround

I decided to add something to the issue template. I'm a rebel like that.

Currently I have a workaround but it feels icky and things get suuuuper slow. Which is saddening but next is so fast when used on its own.

The workaround is quite simple.

  1. Install these deps: npm i --save next-compose-plugins next-transpile-modules

  2. Update next.config.js

// @generated: @expo/next-adapter@2.1.3
// Learn more: https://github.com/expo/expo/blob/master/docs/pages/versions/unversioned/guides/using-nextjs.md#withexpo

const { withExpo } = require('@expo/next-adapter');

const withPlugins = require('next-compose-plugins');
const withTM = require('next-transpile-modules')(['@ui-kitten/components', 'react-native-svg']);

module.exports = withPlugins([withTM], withExpo({
  projectRoot: __dirname,
}));

And now everything works. I used this same workaround with the react-native-web + next template, so I figured I'd give it a go here... And sure enough it works.

created time in 2 months

pull request commentsoftware-mansion/react-native-gesture-handler

fix(web): enable text selection in GestureHandler

@mattmischuk Honestly, I'm using the fork this PR is made from and I disabled gestures on nav for web. It's not pretty but so far it works.

jaulz

comment created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

@andersravn Thought I'd add proof in the shape of a screenshot.

image

Code:

import * as React from 'react'
import { StyleSheet, Text, View } from 'react-native'
import { Button } from '@ui-kitten/components'

export default function App(props) {
  return (
    <View style={styles.container}>
      <Text accessibilityRole="header" style={styles.text}>
        React Native for Web & Next.js
      </Text>

      <Text style={styles.link} accessibilityRole="link" href={`/alternate`}>
        A universal link
      </Text>

      <Button status='primary'>PRIMARY</Button>

      <View style={styles.textContainer}>
        <Text accessibilityRole="header" aria-level="2" style={styles.text}>
          Subheader
        </Text>
      </View>
    </View>
  )
}

const styles = StyleSheet.create({
  container: {
    alignItems: 'center',
    flexGrow: 1,
    justifyContent: 'center',
  },
  link: {
    color: 'blue',
  },
  textContainer: {
    alignItems: 'center',
    marginTop: 16,
  },
  text: {
    alignItems: 'center',
    fontSize: 24,
    marginBottom: 24,
  },
})
andersravn

comment created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

I found the cause. It's because ui-kitten hasn't been transpiled yet. I also found a fix. I don't know if it's a workaround or a solution but this appears to be working for me.

  1. Install these deps:

npm i --save next-compose-plugins next-transpile-modules

  1. Update next.config.js
const withPlugins = require('next-compose-plugins');
const withTM = require('next-transpile-modules')(['@ui-kitten/components', 'react-native-svg']);

module.exports = withPlugins([withTM], {
  webpack: config => {
    config.resolve.alias = {
      ...(config.resolve.alias || {}),
      // Transform all direct `react-native` imports to `react-native-web`
      'react-native$': 'react-native-web',
    }
    config.resolve.extensions = [
      '.web.js',
      '.web.ts',
      '.web.tsx',
      ...config.resolve.extensions,
    ]
    return config
  },
});

From what I've seen you'll have to add all modules that need to be transpiled to the array. It gets a bit slower though, but it's not by a whole lot.

andersravn

comment created time in 2 months

issue commentzeit/next.js

Cannot find module 'react-native' error when trying to use ui-kitten library with react-native-web

3 hours later, I'm facing the same issue. Timing is everything.

andersravn

comment created time in 2 months

issue commentexpo-community/standard-version-expo

Android versionCode bumped to semver?

@byCedric Hey, it's me again. 😄

Prereleases do mess up the ios buildNumber a tad:

"buildNumber": "1.1.0-0",

Is this also a config issue?

RWOverdijk

comment created time in 2 months

issue closedexpo-community/standard-version-expo

Android versionCode bumped to semver?

Description of the bug

standard-version --dry-run

✔ bumping version in package.json from 1.0.0 to 1.1.0 ✔ bumping version in app.json from 1.0.0 to 1.1.0 ✔ bumping version in app.json from 1.0.2 to 1.1.0 ✔ bumping version in app.json from 3 to 1.1.0 ✔ created CHANGELOG.md ✔ outputting changes to CHANGELOG.md

Shouldn't this go from 3 to 4?

To Reproduce

https://dev.to/bycedric/simplify-expo-releases-with-standard-version-2f4o

Expected behavior

For it to go from 3 to 4

Actual behavior

It went from 3 to 1.1.0

Additional context

I know issue is quite empty but... I tried 😅

closed time in 2 months

RWOverdijk

issue commentexpo-community/standard-version-expo

Android versionCode bumped to semver?

That is absolutely correct! It works :) Thanks! 🎉

Maybe update the DEV post as well with the /increment thing for android (I shared it with some people and they'll probably have the same issue).

Again, thanks for this.

RWOverdijk

comment created time in 2 months

issue commentexpo-community/standard-version-expo

Android versionCode bumped to semver?

Maybe relevant, but I'm using these versions:

    "standard-version": "^7.1.0",
    "standard-version-expo": "^1.0.3"
RWOverdijk

comment created time in 2 months

issue commentexpo-community/standard-version-expo

Android versionCode bumped to semver?

@byCedric Same thing. And now I've disabled everything besides the increment for test:

module.exports = {
  bumpFiles: [
    {
      filename: 'app.json',
      updater: require.resolve('standard-version-expo/android/increment'),
    }
  ],
};

Gives me:

standard-version --dry-run "--release-as" "minor"

✔ bumping version in app.json from 3 to 1.1.0 ✔ created CHANGELOG.md ✔ outputting changes to CHANGELOG.md

RWOverdijk

comment created time in 2 months

issue commentexpo-community/standard-version-expo

Android versionCode bumped to semver?

@byCedric aHA! I didn't get that from the dev link I added 😄

"Tries"

But even then, same output.

module.exports = {
  bumpFiles: [
    {
      filename: 'package.json',
    },
    {
      filename: 'app.json',
      updater: require.resolve('standard-version-expo'),
    },
    {
      filename: 'app.json',
      updater: require.resolve('standard-version-expo/ios'),
    },
    {
      filename: 'app.json',
      updater: require.resolve('standard-version-expo/android/increment'),
    },
  ],
};

Gives me:

standard-version --dry-run "--release-as" "prepatch"

✔ bumping version in package.json from 1.0.0 to 1.0.1-0 ✔ bumping version in app.json from 1.0.0 to 1.0.1-0 ✔ bumping version in app.json from 3 to 1.0.1-0 ✔ bumping version in app.json from 1.0.2 to 1.0.1-0 ✔ created CHANGELOG.md ✔ outputting changes to CHANGELOG.md

RWOverdijk

comment created time in 2 months

issue openedexpo-community/standard-version-expo

Android versionCode bumped to semver?

Description of the bug

standard-version --dry-run

✔ bumping version in package.json from 1.0.0 to 1.1.0 ✔ bumping version in app.json from 1.0.0 to 1.1.0 ✔ bumping version in app.json from 1.0.2 to 1.1.0 ✔ bumping version in app.json from 3 to 1.1.0 ✔ created CHANGELOG.md ✔ outputting changes to CHANGELOG.md

Shouldn't this go from 3 to 4?

To Reproduce

https://dev.to/bycedric/simplify-expo-releases-with-standard-version-2f4o

Expected behavior

For it to go from 3 to 4

Actual behavior

It went from 3 to 1.1.0

Additional context

I know issue is quite empty but... I tried 😅

created time in 2 months

more