Description of the bug

I have this extension installed which pulls the workflow schema to auto-completes

but it seems to be wrong

To Reproduce

You need to have this expension installed

https://marketplace.visualstudio.com/items?itemName=cschleiden.vscode-github-actions

Expected behavior

a 1:1 with the auto-suggested (from workflow schema) and the actual allowed commands

Actual behavior

My CI fails

🔖 v2.4.2

Thanks 👍

add space between emoji and number

Minor fix, but it's been bugging me. Adds a space between the count and the emoji. Matching the github style

This is the correct babel config I found:

{
  presets: [
      [
        'babel-preset-expo',
        {
          jsxRuntime: 'classic',
        },
      ],
    ],
}

How did I find it ?

I went in babel-expo-preset and I commented all extra presets and plugins : https://github.com/expo/expo/blob/b783fcc4964b6bfe55ea15893b457a3de5abf3a1/packages/babel-preset-expo/index.js#L47-L116

I uncommented each presets and plugins one by one and found that @babel/plugin-transform-react-jsx was the problem : https://github.com/expo/expo/blob/b783fcc4964b6bfe55ea15893b457a3de5abf3a1/packages/babel-preset-expo/index.js#L47-L58

Changing the value from automatic to classic fixed the issue. I'm still investigating why 🧐

Does this mean that owl work for you with your expo project?

I would be very interested in knowing how you got it working, if you wouldn't mind sharing.

No worries @assayire - I do hope you find the extension useful 🤗

There is no option to report issues on this repo 🤷‍♂️ So will try to use PR instead.. 🙉

The conversation back button is breaking the app. When pressing the back it seems to crash the app in the background and cause it to restart. This breaks some of the cookie auth, so even if the user log in again then it will just spin the loaders forever.

Update README.md

There is no option to report issues on this repo 🤷‍♂️ So will try to use PR instead.. 🙉

Revert "Add FOSSA workflow to enable license scanning"

Merge pull request #82 from intercom/revert-51-add-fossa-workflow

Revert "Add FOSSA workflow to enable license scanning"

Remove deprecated methods from README docs

The docs should be updated with the new suggested alternatives.

I found that I can use the AppState.listener('change', (nextState) => <sign in and out>) for iOS to logout before the user closes the app.

Unlucky this does not work on android. Since closing the app does not cause a AppState "change". Tried listening for the blur but that does not work, since Intercom itself causes the app to blur when showing mobile carousel or conversation.

Is there any way for me to hook into android life-cycle and force a Intercom('shotdown') when the app closes?

Description

I am aware that there is https://hermesengine.dev/docs/language-features

Before Hermes I burned my fingers using notation like 10_000 or Intl. But is there a way for me to be warned by either TS or eslint?

I'm using babel-preset-expo and my tsconfig looks like:

{
  "extends": "expo/tsconfig.base",
  "compilerOptions": {
    "jsx": "react",
    "strict": true,
    "baseUrl": ".",
    "paths": {
      "~/*": ["./*"]
    },
    "types": ["node", "jest"]
  }
}

Should I set the target to ES2015 ???

BTW perhaps it should be considered if Hermes should be listed on MDN like node and deno is

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/at

What about the bright colours from the reactions (👍, ❤️)?

🐛 run on /pull/ url

🔖 v.2.4.1 - fix not showing on PR page

Merge pull request #23 from Norfeldt:fix-PR-page-not-showing-reactions

🔖 v.2.4.1 - fix not showing on PR page

It should be live now 🤘

it does not return a success boolean

Update README code example for .flush

it does not return a success boolean

Chat GPT explains it like this

clearAll(): This method removes all cookies from the cookie store. Any cookies set by your application or by other websites will be removed. This method takes no arguments and returns a promise that resolves when the cookies have been cleared.

flush(): This method saves all cookies to the cookie store immediately. Normally, cookies are saved periodically in the background, but this method forces an immediate save. This method takes no arguments and returns a promise that resolves when the cookies have been saved.

removeSessionCookies(domain: string): This method removes all session cookies for a specific domain. Session cookies are cookies that have no expiration time and are deleted when the user closes the application or the browser. This method takes a single argument, the domain of the cookies to remove, and returns a promise that resolves when the cookies have been removed.

In summary, clearAll removes all cookies, flush saves all cookies immediately, and removeSessionCookies removes only session cookies for a specific domain. It's important to note that these methods operate on the global cookie store, so if you have multiple applications running in the same environment, these methods will affect all of them.

Is this correct?

(I read it like when calling .clearAll() I do not need to call .removeSessionCookies as well?)