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/mockdata #38

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Feat/mockdata #38

wants to merge 4 commits into from

Conversation

djom202
Copy link
Contributor

@djom202 djom202 commented Oct 28, 2024

Issue reference: Mock data

To prevent issues when Apollo is not set up locally and to provide data for E2E tests, we can now test the entire site using mock data without sharing Apollo credentials, ensuring the site functions correctly.

What has changed:

  • I added models to manipulate the data in test mode, allowing us to retrieve information from a JSON file corresponding to the type of data needed.
  • I included NODE_ENV=test as initial data when the repository is cloned.
  • I added random images for the data; however, they can be changed if necessary.

What reviewers should know:

When this change is integrated, it will help complete the E2E tests on components that previously had no data.

Preview

baqjs_page

baqjs_events_page

baqjs_sponsors_page

Copy link

netlify bot commented Oct 28, 2024

👷 Deploy request for tranquil-tartufo-482030 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cba9d6d

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.

1 participant