GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — [email protected] causes issues · Issue #2479 · angular-fullstack/generator-angular-fullstack

This repository was archived by the owner on Jul 29, 2026. It is now read-only.
This repository was archived by the owner on Jul 29, 2026. It is now read-only.

[email protected] causes issues #2479

Copy link
Copy link
Closed
#2498
Closed
[email protected] causes issues#2479
#2498
Copy link
Labels
need details

Description

@sagyeli
sagyeli
opened on Jan 30, 2017
Issue body actions
  • I understand that GitHub issues are not for tech support, but for questions specific to this generator, bug reports, and feature requests.
Item Version
generator-angular-fullstack 4.1.2
Node 4.5.0
npm 3.10.2
Operating System Windows 10
etc etc
Item Answer
Transpiler Babel / TypeScript
Markup HTML / Pug
CSS CSS / LESS / SCSS / Stylus
Router ngRoute / ui-router
Client Tests Jasmine / Mocha
DB MongoDB / SQL
Auth Y / N
etc etc

Please be aware: the following event on thing.events.js does not work: Thing.schema.post(e, emitEvent(event));, since Thing.schema.post event is not triggered with [email protected]

Activity

  1. sagyeli
    changed the title [-][email protected] causes issues with [/-] [+][email protected] causes issues[/+] on Jan 30, 2017
  2. Awk34 commented on Jan 30, 2017

    @Awk34
    Awk34
    on Jan 30, 2017
    Collaborator
    More actions

    Are you sure there is a Mongoose issue here? I don't see anything breaking in the 4.8.0 changelog

  3. Awk34
    added
    need details
    on Jan 30, 2017
  4. gordonaspin commented on Feb 3, 2017

    @gordonaspin
    gordonaspin
    on Feb 3, 2017
    More actions

    +1 I had same problem with Thing.schema.post(e, emitEvent(event)) not emitting events in mongoose 4.8.1, downgraded to mongoose 4.7.7 and everything ok. Also, I would suggest adding ghostMode: false to the browsersync config in server/config/express.js as the default.

  5. Awk34 commented on Feb 3, 2017

    @Awk34
    Awk34
    on Feb 3, 2017
    Collaborator
    More actions

    Have either of you submitted a bug report to Mongoose yet?

  6. gordonaspin commented on Feb 3, 2017

    @gordonaspin
    gordonaspin
    on Feb 3, 2017 via email
    More actions
    I did not. I wasn’t sure it was mongoose at fault or not. I’m kind of new to fullstack and not confident I could pull together a stand-alone example of the event emitting not working.
    …
    On Feb 3, 2017, at 9:12 AM, Andrew Koroluk ***@***.***> wrote: Have either of you submitted a bug report to Mongoose yet? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2479 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AJ3IEGSl9UAnuxYHSRoR0-Rv3v8QO0l2ks5rY0PggaJpZM4LxL_B>.
  7. Awk34 commented on Feb 3, 2017

    @Awk34
    Awk34
    on Feb 3, 2017
    Collaborator
    More actions

    From reading this bug report, it seems like there might have been an unintended breaking API change in mongoose 4.8.0

    Reacted by Eli Krumholz
  8. EliKrumholz commented on Feb 5, 2017

    @EliKrumholz
    EliKrumholz
    on Feb 5, 2017
    More actions

    I think I was able to replicate the bug, I opened an issue with mongoose: Automattic/mongoose#4955

  9. EliKrumholz commented on Feb 7, 2017

    @EliKrumholz
    EliKrumholz
    on Feb 7, 2017
    More actions

    From the Mongoose folks:

    mutating the underlying schema after the Thing model has been created by mongoose.model() isn't a supported behavior. Make sure you define all your hooks before calling mongoose.model()

    @ernisto has a solution in #2491 that does just that.

  10. alkiko commented on Feb 7, 2017

    @alkiko
    alkiko
    on Feb 7, 2017
    More actions

    Downgrading to 4.7.9 solved it for me too, but @ernisto's suggestion looks more correct.

  11. Awk34 commented on Feb 7, 2017

    @Awk34
    Awk34
    on Feb 7, 2017
    Collaborator
    More actions

    @ernisto do you thing you might submit a Pull Request for your fix?

  12. Climax777 commented on Feb 8, 2017

    @Climax777
    Climax777
    on Feb 8, 2017
    More actions

    +1 for the @ernisto fix

  13. ernistkg commented on Feb 9, 2017

    @ernistkg
    ernistkg
    on Feb 9, 2017
    More actions

    @Awk34 Yes it would be Great! But I need some time to create Pull Request according to rules and best practices.

  14. kingcody
    added a commit that references this issue on Feb 11, 2017

    fix(events): mongoose event registration

    This issue will close once commit e3ed67f is merged into the 'canary' branch.
    e3ed67f
  15. Awk34
    closed this as completedin #2498on Feb 11, 2017
  16. Awk34
    added 2 commits that reference this issue on May 13, 2017

    fix(events): mongoose event registration

    This issue will close once commit f69f560 is merged into the 'master' branch.
    f69f560

    fix(events): mongoose event registration

    This issue will close once commit 79bdeed is merged into the 'master' branch.
    79bdeed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    need details

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.