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

feat(#458): Add DruxtAuth to DruxtSite example #547

Merged
merged 5 commits into from
Aug 9, 2022

Conversation

Decipher
Copy link
Member

@Decipher Decipher commented Aug 7, 2022

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly. (PR: #)
  • I have added tests to cover my changes (if not applicable, please state why)
  • All new and existing tests are passing.

Testing:

In Gitpod:

  1. Open PR in Gitpod
  2. Run yarn example:druxt-site
  3. Open site (CTRL+Click localhost:3000 link)
  4. Click Login
  5. Login to Drupal (Use onetime login link above Druxt welcome text, or cd docs/drupal && ddev drush uli)
  6. Grant access
  7. Inspect the Authenticated user in console: this.$nuxt.$auth.user

Screenshots/Media:

@changeset-bot
Copy link

changeset-bot bot commented Aug 7, 2022

⚠️ No Changeset found

Latest commit: 0c6cad9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov
Copy link

codecov bot commented Aug 7, 2022

Codecov Report

Merging #547 (0c6cad9) into develop (da58130) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #547   +/-   ##
========================================
  Coverage    96.60%   96.60%           
========================================
  Files           86       86           
  Lines         2208     2208           
  Branches       433      433           
========================================
  Hits          2133     2133           
  Misses          68       68           
  Partials         7        7           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@Decipher Decipher mentioned this pull request Aug 7, 2022
18 tasks
.gitpod.yml Show resolved Hide resolved
.gitpod.yml Show resolved Hide resolved
@Decipher Decipher merged commit 072a715 into develop Aug 9, 2022
@Decipher Decipher deleted the feature/458-druxt_site_auth branch August 9, 2022 22:22
Decipher added a commit that referenced this pull request Oct 27, 2022
Decipher added a commit that referenced this pull request Oct 28, 2022
* feat(#578): add DruxtDevelTemplate component

* feat(#578): update missing template message

* chore(#578): update template formatting

* chore(#578): fix broken test

* feat(#578): update missing template message

* chore: update entity mixin

* feat(#578): update missing default slot message

* chore(#578): validate path

* chore(#578): update test coverage

* chore(#578): add rate limiter

* chore: fix breaking test

* chore(#578): add test coverage for template tool

* chore(#578): update ci

* chore(#578): update slot message and description

* feat(#578): added support for devel template tool

* chore(#578): update template string validation

* chore(#578): add conditional defaults

* feat(#578): add template support to block region

* feat(#578): add support for template tool

* feat(#578): update debug value

* feat(#578): add support for template tool

* feat(#578): add support for the template tool

* feat(#547): add support for the template tool

* chore(#578): fix router slots
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.

Create Nuxt auth example
1 participant