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

[Master] Add readme documentation for IStorageBuffer #2045

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Conversation

MSNev
Copy link
Collaborator

@MSNev MSNev commented Apr 11, 2023

No description provided.

@MSNev MSNev added this to the 2.8.12 milestone Apr 11, 2023
@@ -233,7 +233,7 @@
});

const useSessionStorage = _self._senderConfig.enableSessionStorageBuffer() &&
(_self._senderConfig.bufferOverride() || utlCanUseSessionStorage())
!!(_self._senderConfig.bufferOverride() || utlCanUseSessionStorage())

Check notice

Code scanning / CodeQL

Semicolon insertion

Avoid automated semicolon insertion (96% of all statements in [the enclosing function](1) have an explicit semicolon).
@MSNev MSNev merged commit d256816 into master Apr 11, 2023
MSNev added a commit that referenced this pull request Apr 12, 2023
* [BUG] The documentation for enableDebug is incorrect, it should reference enableDebugExceptions #2014 (#2022)

* [BUG] SDK LOAD Failure reporting not working #2027 (#2038)

* Add a simple interface to enable custom buffer storage solutions #1419 (#2037)

This can be used like:

```
const appInsights = new ApplicationInsights({
  config: {
    enableSessionStorageBuffer: true,
    bufferOverride: {
      getItem: (logger, key) => localStorage.getItem(key),
      setItem: (logger, key, value) => localStorage.setItem(key, value),
    }
  }
});
```

Co-authored-by: Nev <54870357+MSNev@users.noreply.github.com>

* [Master] Add readme documentation for IStorageBuffer (#2045)

* [Release] Increase version to 2.8.12 (#2046)

---------

Co-authored-by: Philip Peitsch <philip.peitsch@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants