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

Integrate LFX header #1148

Merged
merged 5 commits into from
Jul 20, 2023
Merged

Integrate LFX header #1148

merged 5 commits into from
Jul 20, 2023

Conversation

joanagmaia
Copy link
Contributor

@joanagmaia joanagmaia commented Jul 20, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at ae11529

This pull request adds a new LFX header component to the frontend of the application, and removes or simplifies some components and code related to the previous authentication and profile editing functionality. The LFX header component provides a consistent navigation and branding across different LFX products, and integrates with the Auth0 service for user authentication and profile management. The pull request affects several files in the frontend/src directory, as well as the frontend/index.html file.

🤖 Generated by Copilot at ae11529

LFX header added
Auth0Service improved
Winter code cleanup

Why

How

🤖 Generated by Copilot at ae11529

  • Add LFX header component to the application to provide consistent navigation and branding across LFX products (link, link, link, link, link, link, link, link, link, link)
  • Remove unused or redundant components and routes related to user profile, sign-in, and account options, as they are handled by the LFX header component (link, link, link, link, link, link, link, link, link, link, link, link)
  • Remove unnecessary console.log statement from auth-service.js (link)
  • Remove empty line from auth-guard.js (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@joanagmaia joanagmaia added the Feature Created by Linear-GitHub Sync label Jul 20, 2023
@joanagmaia joanagmaia self-assigned this Jul 20, 2023
@joanagmaia joanagmaia merged commit 6d1c463 into crowd-linux Jul 20, 2023
5 checks passed
@joanagmaia joanagmaia deleted the feature/lfx-header branch July 20, 2023 13:14
joanagmaia added a commit that referenced this pull request Jul 21, 2023
joanagmaia added a commit that referenced this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants