Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

Chain promises together in lib/browser.ts #3904

Closed
Jakub41/Flights-Scraper-CPH-Kastrup
#46
Closed
Chain promises together in lib/browser.ts#3904
Jakub41/Flights-Scraper-CPH-Kastrup
#46
Labels
hotlist: polishstatus: pr pendingtype: cleanup
@sjelin

Description

@sjelin
sjelin
opened on Dec 30, 2016

There are a few places in lib/browser.ts where we should have chained promises together, but didn't because the control flow handled it for us:

  • In ptorMixin, if setupFn() has a then
  • In .get
  • In .setLocation
  • In .getLocationAbsUrl
  • In .debugger

I may add to this list if I find more.

Metadata

Metadata

Assignees

No one assigned

    Labels

    hotlist: polishstatus: pr pendingtype: cleanup

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions