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

[AB3: Removing Fields] Git diff for removing address field #84

Open
mfjkri opened this issue Sep 23, 2024 · 2 comments
Open

[AB3: Removing Fields] Git diff for removing address field #84

mfjkri opened this issue Sep 23, 2024 · 2 comments

Comments

@mfjkri
Copy link
Contributor

mfjkri commented Sep 23, 2024

Situation

Currently, there is no git diff reflecting the deletion of the address field, which may cause confusion for students.

Problem

Without this information, students might be uncertain if they have correctly deleted the address field as required.

Proposed Solution

Create a commit specifically for deleting the address field and include the corresponding git diff into the guide.

I believe consolidating all the changes into a single commit would be sufficient for this purpose.

@mfjkri mfjkri changed the title [AB3: Removing Fields] Git diff [AB3: Removing Fields] Git diff for removing address field Sep 23, 2024
@damithc
Copy link
Contributor

damithc commented Sep 23, 2024

@mfjkri Yes, this could be useful.

@mfjkri
Copy link
Contributor Author

mfjkri commented Sep 23, 2024

Alright I will start working on this

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 a pull request may close this issue.

2 participants