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

Update Preferences state #21

Merged
merged 1 commit into from
Nov 13, 2018
Merged

Update Preferences state #21

merged 1 commit into from
Nov 13, 2018

Conversation

estebanmino
Copy link
Contributor

This PR fixes some Preferences Controller state updating inside GABA but not in the general state.

@codecov-io
Copy link

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #21   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          18     18           
  Lines         889    889           
  Branches       97     97           
=====================================
  Hits          889    889
Impacted Files Coverage Δ
src/PreferencesController.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb6f68b...9ca091a. Read the comment docs.

Copy link
Contributor

@bitpshr bitpshr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, looks good.

@bitpshr bitpshr merged commit 6e9a32d into master Nov 13, 2018
@bitpshr bitpshr deleted the update-preferences-state branch November 13, 2018 17:57
rickycodes added a commit that referenced this pull request Nov 23, 2022
* 32.0.3

* update changelog

* use marjor

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Ricky Miller <[email protected]>
mcmire added a commit to mcmire/core that referenced this pull request Jul 17, 2023
Bump TypeScript and fix compile errors. Specifically, an error that's
caught is now typed as `unknown`, so accessing `message` on that error
object won't work anymore. This is fixed using a type guard.

Also bump `typedoc` to fix peer dependency warnings.
MajorLift pushed a commit that referenced this pull request Sep 22, 2023

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: legobt <[email protected]>
kanthesha pushed a commit that referenced this pull request Oct 11, 2023
* 5.1.6; update batch request handling

* remove Promise wrapping

* simplify _handleBatch implementation

* add clarifying comments

* standardize internal errors

Co-authored-by: Dan Finlay <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants