Skip to content

Gathepress venue #968

Merged
mauteri merged 10 commits intoGatherPress:developfrom
prashantabellad:gathepress_venue_map
Jan 6, 2025
Merged

Gathepress venue #968
mauteri merged 10 commits intoGatherPress:developfrom
prashantabellad:gathepress_venue_map

Conversation

@Prayag11
Copy link
Copy Markdown
Contributor

Description of the Change

Closes #

How to test the Change

Changelog Entry

Added - New feature
Changed - Existing functionality
Deprecated - Soon-to-be removed feature
Removed - Feature
Fixed - Bug fix
Security - Vulnerability

Credits

Props @username, @username2, ...

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Dec 24, 2024

PR Summary

  • New Test File for Venue Map Verification
    A new end-to-end test file has been added that aids in verifying the venue map on the admin side.

  • Username Update in Tests
    The username in the existing tests has been updated from 'testuser1' to 'prashantbellad' for better reliability.

  • New Test File for Event Location Visibility Verification
    A new test has been introduced to verify the visibility of the event location in the venue post.

  • Tests to Check Venue Map Visibility with Toggle Switch
    New tests have been added that evaluate the visibility of the venue map when the toggle switch is turned ON and OFF.

  • Minimal Additions in the Event Creation Flow
    Minor adjustments have been made in the event creation flow in the gatherpress-event-loggedIn.spec.js file to enhance the user experience.

  • Use of .js Extension in Module Imports
    To maintain consistency, module imports in multiple files now use the .js extension.

  • Map Visibility Verification after Event Creation
    A new test has been added to verify the visibility of the map following the creation of the event.

  • Modification in Venue Selector Option Passing
    The method of passing the venue Selector option in event tests has been modified for events that have been newly created. This optimizes the system’s overall functionality.

@mauteri
Copy link
Copy Markdown
Contributor

mauteri commented Dec 24, 2024

@Prayag11 you can run npm run lint:js:fix to fix JS issues automatically. You can also run npm run lint:js to just see what the issues are so you can fix some manually that can't be fixed automatically. All of the commands are listed in package.json that can help you with development. Thanks!

@Prayag11
Copy link
Copy Markdown
Contributor Author

Prayag11 commented Jan 2, 2025

@Prayag11
Copy link
Copy Markdown
Contributor Author

Prayag11 commented Jan 2, 2025

Verify.the.offline.venue.location.map.should.not.be.visible.on.the.venue.post.when.the.display.map.toggled.button.is.disabled.webm

@Prayag11
Copy link
Copy Markdown
Contributor Author

Prayag11 commented Jan 2, 2025

@Prayag11
Copy link
Copy Markdown
Contributor Author

Prayag11 commented Jan 2, 2025

Verify.the.event.post.event.details.and.timezone.should.be.visible.on.the.front.end.webm

@mauteri mauteri merged commit 0137858 into GatherPress:develop Jan 6, 2025
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.

2 participants