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

Production server e2e #7

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Production server e2e #7

wants to merge 13 commits into from

Conversation

jtankw3
Copy link
Owner

@jtankw3 jtankw3 commented Nov 2, 2020

Fixes #

PR Checklist

Ensure that you have:

  • Read and understood our PR guideline: https://github.com/TEAMMATES/teammates/blob/master/docs/process.md#step-4-submit-a-pr
    • Added the issue number to the "Fixes" keyword above
    • Titled the PR as specified in the abovementioned document
  • Made your changes on a branch other than master and release
  • Gone through all the changes in this PR and ensured that:
    • They addressed one (and only one) issue
    • No unintended changes were made
  • Run and passed static analysis: ./gradlew lint and npm run lint
  • Added/updated tests, if changes in functionality were involved
  • Added/updated documentation to public APIs (classes, methods, variables), if applicable

Outline of Solution

@jtankw3 jtankw3 changed the base branch from instructor-result-e2e to master November 2, 2020 14:02
…production-server-e2e

# Conflicts:
#	src/e2e/java/teammates/e2e/cases/FeedbackSubmitPageE2ETest.java
#	src/e2e/java/teammates/e2e/cases/InstructorCourseDetailsPageE2ETest.java
#	src/e2e/java/teammates/e2e/cases/InstructorFeedbackSessionsPageE2ETest.java
#	src/e2e/java/teammates/e2e/cases/InstructorHomePageE2ETest.java
@jtankw3 jtankw3 force-pushed the production-server-e2e branch from a4a3dec to ceeeaff Compare November 8, 2020 18:24
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