-
Notifications
You must be signed in to change notification settings - Fork 34
Add UI for adding catalog numbers to a series #1341
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
Labels
Milestone
Comments
@ArtemFedorchuk Hi, please, let me know if you need help or if you aren't going to work on this issue. Thanks! |
We can copy&paste code from #1342 |
bahoss
added a commit
to bahoss/mystamps
that referenced
this issue
May 13, 2020
bahoss
added a commit
to bahoss/mystamps
that referenced
this issue
May 14, 2020
bahoss
added a commit
to bahoss/mystamps
that referenced
this issue
May 19, 2020
bahoss
added a commit
to bahoss/mystamps
that referenced
this issue
May 20, 2020
bahoss
added a commit
to bahoss/mystamps
that referenced
this issue
May 20, 2020
php-coder
added a commit
that referenced
this issue
Jun 13, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
On the
src/main/webapp/WEB-INF/views/series/info.html
page we should add another form to add catalog numbers. It should have inputs for numbers (just a text input) and a catalog name. Also there should be a submit button. Here are examples that can be helpful:List of catalogs:
mystamps/src/main/webapp/WEB-INF/views/site/index.html
Lines 174 to 184 in 366f099
Catalog numbers:
mystamps/src/main/webapp/WEB-INF/views/series/add.html
Lines 378 to 399 in 366f099
When we submit a form, it should send a PATCH request to API
/series/{id}
withand reload a page on success. The value of
path
depends on a value of catalog name -- for Michel catalog, we use "/michel_numbers" but for Scott it should be "/scott_numbers" and so on.It should show an error when server returns an error. Note that there can be 2 type of errors -- general (for a whole form) and validation error (for a particular field).
Part of #1339
The text was updated successfully, but these errors were encountered: