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

push your code #98

Open
wants to merge 132 commits into
base: List
Choose a base branch
from
Open

push your code #98

wants to merge 132 commits into from

Conversation

safayoussef
Copy link
Contributor

@safayoussef safayoussef commented Oct 31, 2022

☑️ Definition of Done checklist

  • Meaningful title and valuable description in pull request
  • Code change tested manually if not covered by automated tests
  • Pull request linked to one or more issues/stories
  • Acceptance criteria of the linked stories satisfied
  • Code change comprehended by those who approved it
  • State of target branch maintained or improved after the code change
  • Go to MainActivityTest in the Android App and uncommit @test
    Run test manually. Add a picture of the test result in the comments.
    Comment @test again to avoid future problems with the CI

Quinstedt and others added 30 commits May 18, 2022 10:53
This commit enables player name and score to be saved in a SQL database.
Then,the data can be retrived from the database to show it on the Leaderboard.
The leaderboard also has a function to clear all player scores.

Related Issues: #4 , #30 , #71
Since, the server will be removed. The game server workflow is not needed anymore.
The server directory is no longer needed since we are introducing the leaderboard inside the database.
The parameter "leaderboard.class" is replaced now with "Scoreboard.class"
Set export Schema to false for to avoid android CI error
…creen' into Database-Setup-and-Leaderboard-Screen
…creen' into Database-Setup-and-Leaderboard-Screen
Your changes needs to be commited if you want to change branch and avoid losing the changes
Closes related issue: #6
# Conflicts:
#	Android/islandRush/app/src/androidTest/java/com/quinstedt/islandRush/LeaderboardTest.kt
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/BrokerConnection.java
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/ControlPad.java
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/Joystick.java
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/MainActivity.java
The logic for save score screen is now implemented in the leaderboard animation class. The leaderboard activity and the save score screen is deleted. The position of number 5 is fixed.
The logic for save score screen is now implemented in the leaderboard animation class. The leaderboard activity and the save score screen is deleted. The position of number 5 is fixed.
This reverts commit 854be02.
…he animation and in the leaderbord and stored in the database.
faisal1201 and others added 30 commits May 28, 2022 02:27
Modding Instruction Removed
Add annotation for camera in Smartcar Specification
# Conflicts:
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/activityClasses/ControlChoice.java
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/activityClasses/ControlPad.java
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/activityClasses/Joystick.java
…PLAY_SCREEN

# Conflicts:
#	Android/islandRush/app/src/main/java/com/quinstedt/islandRush/activityClasses/ControlChoice.java
How to play Instructions and button Animation is approved
+ made a new mod with different starting point and option to get to more places on the island and on the other islands
Use 'actions/upload-artifact@v2' to try to export without zipping
+ removed Exploration.zip file
Final Change. Waiting for the video to be inserted.
Small fixes
Closes related issue: #90
Closes related issue: #91
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.

4 participants